git: 4d9eaa04bde6 - stable/15 - arm64/iommu: Fix a resource leak in smmu_domain_alloc()

The branch stable/15 has been updated by markj:

URL: https://cgit.FreeBSD.org/src/commit/?id=4d9eaa04bde65cf6ed19bc801db1f2a28f89639e

commit 4d9eaa04bde65cf6ed19bc801db1f2a28f89639e
Author:     Mark Johnston <markj@FreeBSD.org>
AuthorDate: 2026-01-15 14:04:19 +0000
Commit:     Mark Johnston <markj@FreeBSD.org>
CommitDate: 2026-01-26 00:51:44 +0000

    arm64/iommu: Fix a resource leak in smmu_domain_alloc()
    
    We should free the allocated ASID if smmu_init_cd() fails.
    
    Move the allocation of "domain" to simplify the first error path.
    
    Reported by:    Kevin Day <kevin@your.org>
    Reviewed by:    br
    MFC after:      1 week
    Differential Revision:  https://reviews.freebsd.org/D54676
    
    (cherry picked from commit 6740cccb1eff2a0e1e6d451fa9676a21736937d2)
---
 sys/arm64/iommu/smmu.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/sys/arm64/iommu/smmu.c b/sys/arm64/iommu/smmu.c
index ec8e04ce117b..a84ef4ae347e 100644
--- a/sys/arm64/iommu/smmu.c
+++ b/sys/arm64/iommu/smmu.c
@@ -1698,22 +1698,21 @@ smmu_domain_alloc(device_t dev, struct iommu_unit *iommu)
 
 	unit = (struct smmu_unit *)iommu;
 
-	domain = malloc(sizeof(*domain), M_SMMU, M_WAITOK | M_ZERO);
-
 	error = smmu_asid_alloc(sc, &new_asid);
 	if (error) {
-		free(domain, M_SMMU);
 		device_printf(sc->dev,
 		    "Could not allocate ASID for a new domain.\n");
 		return (NULL);
 	}
 
+	domain = malloc(sizeof(*domain), M_SMMU, M_WAITOK | M_ZERO);
 	domain->asid = (uint16_t)new_asid;
 
 	smmu_pmap_pinit(&domain->p);
 
 	error = smmu_init_cd(sc, domain);
 	if (error) {
+		smmu_asid_free(sc, domain->asid);
 		free(domain, M_SMMU);
 		device_printf(sc->dev, "Could not initialize CD\n");
 		return (NULL);