[SSSD] [PATCHES] Enable printf format string checking

Ondrej Kos okos at redhat.com
Wed Jul 24 07:41:12 UTC 2013


On 07/23/2013 06:21 PM, Lukas Slebodnik wrote:
> ehlo,
>
> It would be great to have enabled printf format string checking in RHEL7.
> Therefore I decided to send pateches for ticket
> https://fedorahosted.org/sssd/ticket/1945
>
> Patch 0001 -- Even if this patch is first, it should be applied in upstream as
>           last.
>
> Patch 0002 - 0006 -- trivial
> Patch 0007 -- ssize_t is not defined in ANSI c99, IIRC it is defined in posix
>                as a signed type, therefore format should be used "zd"
> Patch 0008 -- size_t is defined in ANSI c99 as unzigned type -> "zu"
> Patch 0009 -- wrapper for inttypes.h and for future format macros.
> Patch 0012 -- formating types defined in stdint.h uint_32_t ...
>             -- there are used macros defined in inttypes.h
> Patches 0010 - 0015 fix formating for special variables (key_serial_t, rlim_t...)
>                      and for some types I created macros in sss_format.h
> Patch 0010 -- key_serial_t is typedef of int32_t, but it could be defined
>           differently in another (platforms/ implemantations of kerberos)
> Patch 0011 -- it seems that rlim_t us the same as uint64_t, but it was defined
>             using conditional build an not a typedef of std types.
> Patch 0013 -- time_t is defined as "long int" sizeof_i386(time_t) != sizeof_x86_64(time_t)
> Patch 0014 -- ber_int and ber_tag are typedef for int and unsigned long
> Patch 0015 -- gid_t and uid_t are typedef as unsigned
>                (I checked linux 32 bit, linu 64_bit and freebsd 64 bit)
> Patch 0016 -- trivial
>
> LS
>
>
Hi,

I'm looking at the patches now, and I see, that somewhere you changed 
the debug levels to the new style and somewhere you left the numeric 
value. Could you please replace the debug levels with new style anywhere 
you're modifying code?

Ondra


-- 
Ondrej Kos
Associate Software Engineer
Identity Management - SSSD
Red Hat Czech



More information about the sssd-devel mailing list