Re: [Xen-devel] [PATCH v7 2/8] libxl: introduce a new structure to represent static shared memory regions

[Wei Liu <wei.liu2@xxxxxxxxxx>]

On Wed, Sep 12, 2018 at 11:29:32AM +0100, Julien Grall wrote:
> 
> 
> On 09/12/2018 12:09 AM, Stefano Stabellini wrote:
> > > > +
> > > > +/* Constants for libxl_static_shm */
> > > > +#define LIBXL_SSHM_RANGE_UNKNOWN UINT64_MAX
> > > > +#define LIBXL_SSHM_ID_MAXLEN    128
> > > 
> > > Why do you need to bound the size of the string?
> > 
> > It is passed by the user, it is good to be clear about the max size?
> 
> I wasn't able to find other string in libxl to be bound. So I am not sure
> why this one would be. Maybe Wei knows?

String is not bounded in libxl. But I think it is fair enough if we
restrict the length in code.

(Sorry for the very late reply)

Wei.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel