Re: [PATCH 1/1] libselinux: selinux_restorecon: fix printf format string specifier for uint64_t

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Nov 6, 2018 at 5:11 PM Stephen Smalley <sds@xxxxxxxxxxxxx> wrote:
>
> On 11/5/18 3:57 PM, Nicolas Iooss wrote:
> > fc_count is defined as uint64_t, which needs to be printed with PRIu64
> > (it is "llu" on x86 and "lu" on x86-64). Otherwise, building with
> > 'CC="gcc -m32"' fails with:
> >
> >      selinux_restorecon.c: In function ‘restorecon_sb’:
> >      selinux_restorecon.c:633:26: error: format ‘%lu’ expects argument of
> >      type ‘long unsigned int’, but argument 3 has type ‘uint64_t’ {aka
> >      ‘long long unsigned int’} [-Werror=format=]
> >           fprintf(stdout, "\r%luk", fc_count / STAR_COUNT);
> >                              ~~^
> >                              %llu
> >
> > Signed-off-by: Nicolas Iooss <nicolas.iooss@xxxxxxx>
>
> Acked-by: Stephen Smalley <sds@xxxxxxxxxxxxx>

Merged.

Nicolas




[Index of Archives]     [Selinux Refpolicy]     [Linux SGX]     [Fedora Users]     [Fedora Desktop]     [Yosemite Photos]     [Yosemite Camping]     [Yosemite Campsites]     [KDE Users]     [Gnome Users]

  Powered by Linux