On Tue, 17 Dec 2013, Peter Maydell wrote:
> On 17 December 2013 17:30, Stefano Stabellini
> <stefano.stabell...@eu.citrix.com> wrote:
> > Signed-off-by: Stefano Stabellini <stefano.stabell...@eu.citrix.com>
> > ---
> > hw/xen/xen_backend.c | 18 ++++++++++++++++++
> > include/hw/xen/xen_backend.h | 2 ++
> > 2 files changed, 20 insertions(+)
> >
> > diff --git a/hw/xen/xen_backend.c b/hw/xen/xen_backend.c
> > index 197795f..5f9be5a 100644
> > --- a/hw/xen/xen_backend.c
> > +++ b/hw/xen/xen_backend.c
> > @@ -112,6 +112,19 @@ int xenstore_read_int(const char *base, const char
> > *node, int *ival)
> > return rc;
> > }
> >
> > +int xenstore_read_uint64(const char *base, const char *node, uint64_t
> > *uval)
> > +{
> > + char *val;
> > + int rc = -1;
> > +
> > + val = xenstore_read_str(base, node);
> > + if (val && 1 == sscanf(val, "%"PRIu64, uval)) {
>
> PRIxxx are printf format macros, not scanf format ones. You want
> SCNu64
>
> http://pubs.opengroup.org/onlinepubs/9699919799/basedefs/inttypes.h.html
Well spotted, thanks.
