From: mike <mike@7a7537e8-13f0-0310-91df-b6672ffda945>
Date: Wed, 2 Nov 2011 15:45:35 +0000 (+0000)
Subject: The scheduler incorrectly used free() on a POSIX ACL value, which could cause a
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=960c4658e6477dbc51e2c17fb918c11de15e57bb;p=thirdparty%2Fcups.git

The scheduler incorrectly used free() on a POSIX ACL value, which could cause a
crash (STR #3970)

Still investigating why the reporter is getting an ACL validation error
(necessary for this code path to execute)



git-svn-id: svn+ssh://src.apple.com/svn/cups/cups.org/trunk@10098 7a7537e8-13f0-0310-91df-b6672ffda945
---

diff --git a/CHANGES-1.5.txt b/CHANGES-1.5.txt
index d2ddac4a35..871542c222 100644
--- a/CHANGES-1.5.txt
+++ b/CHANGES-1.5.txt
@@ -5,6 +5,8 @@ CHANGES IN CUPS V1.5.1
 
 	- Documentation updates (STR #3885, STR #3946)
 	- Build fixes (STR #3956)
+	- The scheduler incorrectly used free() on a POSIX ACL value, which
+	  could cause a crash (STR #3970)
 	- PPD files using the MacStandard encoding did not work.
 	- The web interface did not work on some platforms (STR #3902)
 	- The lpstat command would crash when then "-u" option was used by a
diff --git a/scheduler/cert.c b/scheduler/cert.c
index d3ef7fdb61..087e2564d2 100644
--- a/scheduler/cert.c
+++ b/scheduler/cert.c
@@ -206,7 +206,7 @@ cupsdAddCert(int        pid,		/* I - Process ID */
 	  *textptr = ',';
 
 	cupsdLogMessage(CUPSD_LOG_ERROR, "ACL: %s", text);
-	free(text);
+	acl_free(text);
       }
 #  endif /* HAVE_MBR_UID_TO_UUID */