Add printf format checks to afs_com_err()
authorSimon Wilkinson <sxw@inf.ed.ac.uk>
Sat, 7 Nov 2009 22:31:08 +0000 (22:31 +0000)
committerDerrick Brashear <shadow|account-1000005@unknown>
Wed, 3 Feb 2010 20:34:12 +0000 (12:34 -0800)
commit8229e668deee3eb00a295a8c9ea96a66b7049687
tree5a9e67f8c439a3c5bb1465a084c5020e017f7a77
parent68463b6ab9c664303692ac264871723c27bfc524
Add printf format checks to afs_com_err()

Add gcc printf format checks to the afs_com_err() functions

Deal with the fallout, in particular change callers which pass
an empty format string to pass NULL instead - the com_err functions
already permit this alternate use.

There's a couple of real bugs here - in one case, we attempt to
print a NULL pointer, rather than a security index, and in the other
we supply a NULL format string, rather than the string we meant to
print.

Change-Id: Icd48f92a4447d4af3dba9a4caa2ff73c1657ad47
Reviewed-on: http://gerrit.openafs.org/794
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
src/aklog/klog.c
src/bucoord/commands.c
src/bucoord/dump.c
src/bucoord/restore.c
src/bucoord/ubik_db_if.c
src/comerr/com_err.h
src/kauth/admin_tools.c
src/ptserver/db_verify.c
src/uss/uss_kauth.c