No need to keep warning about ACLs

[michaelrsweet]

Version: 1.2rc1
CUPS.org User: twaugh.redhat

When /var/run/cups/certs/0 is on a filesystem that does not support POSIX ACLs, each time the certificate is renewed there is a new line added to error_log:

Unable to set ACLs on root certificate "/var/run/cups/certs/0" - Operation not supported

It's probably sufficient just to warn once, as in the attached patch.

[michaelrsweet]

CUPS.org User: mike

Commited with some formatting changes (to conform to our coding standards...)

[michaelrsweet]

"cups-acls.patch":

--- cups-1.2rc1/scheduler/cert.c.acls 2006-04-03 11:22:52.000000000 +0100
+++ cups-1.2rc1/scheduler/cert.c 2006-04-03 12:58:21.000000000 +0100
@@ -216,10 +216,16 @@
}

endif /* HAVE_MBR_UID_TO_UUID */

•  if (acl_set_fd(fd, acl))
  

• cupsdLogMessage(CUPSD_LOG_ERROR,

•       "Unable to set ACLs on root certificate \"%s\" - %s",
  

•       filename, strerror(errno));
  

•  if (acl_set_fd(fd, acl)) {
  

• static int acls_not_supported = 0; /* Only warn once */
• if (errno != EOPNOTSUPP || !acls_not_supported)

• cupsdLogMessage(CUPSD_LOG_ERROR,
  

•         "Unable to set ACLs on root certificate \"%s\" - %s",
  

•         filename, strerror(errno));
  

• if (errno == EOPNOTSUPP)

• acls_not_supported = 1;
  

•  }
  

  • acl_free(acl);
    }
    #endif /* HAVE_ACL_INIT */