On Mon, Nov 5, 2018 at 9:55 PM Nicolas Iooss <nicolas.iooss@xxxxxxx> wrote: > > On Mon, Nov 5, 2018 at 3:36 PM Stephen Smalley <sds@xxxxxxxxxxxxx> wrote: > > > > Use PRIx64 to print the subnet_prefix (which is a uint64_t) instead > > of lx. > > > > Fixes https://github.com/SELinuxProject/selinux/issues/108 > > > > Signed-off-by: Stephen Smalley <sds@xxxxxxxxxxxxx> > > --- > > libsepol/src/ibpkeys.c | 5 +++-- > > 1 file changed, 3 insertions(+), 2 deletions(-) > > > > diff --git a/libsepol/src/ibpkeys.c b/libsepol/src/ibpkeys.c > > index 682bf5db..68a9bdfe 100644 > > --- a/libsepol/src/ibpkeys.c > > +++ b/libsepol/src/ibpkeys.c > > @@ -1,5 +1,6 @@ > > #include <netinet/in.h> > > #include <stdlib.h> > > +#include <inttypes.h> > > > > #include "debug.h" > > #include "context.h" > > @@ -176,7 +177,7 @@ int sepol_ibpkey_query(sepol_handle_t *handle, > > return STATUS_SUCCESS; > > > > err: > > - ERR(handle, "could not query ibpkey subnet prefix: %#lx range %u - %u exists", > > + ERR(handle, "could not query ibpkey subnet prefix: %#" PRIx64 " range %u - %u exists", > > subnet_prefix, low, high); > > return STATUS_ERR; > > } > > @@ -203,7 +204,7 @@ int sepol_ibpkey_modify(sepol_handle_t *handle, > > return STATUS_SUCCESS; > > > > err: > > - ERR(handle, "could not load ibpkey subnet prefix: %#lx range %u - %u exists", > > + ERR(handle, "could not load ibpkey subnet prefix: %#" PRIx64 " range %u - %u exists", > > subnet_prefix, low, high); > > if (ibpkey) { > > context_destroy(&ibpkey->context[0]); > > -- > > 2.19.1 > > > > This looks good to me, and while testing some things, I found a > similar issue in libselinux/src/selinux_restorecon.c. Patch > incoming... > > Cheers, > Nicolas Merged. Nicolas
