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 d2ddac4a35c3abc4e2936fd9808c5a20c8c2b8e9..871542c222f6e800d54f5fe7777bf53cad149fee 100644 (file)
--- 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 d3ef7fdb618abf00cfe105539a8515031b5050dd..087e2564d20fcf5774a41149d612f808cc738d7e 100644 (file)
--- 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 */