On 27.03.2017 15:26, Markus Armbruster wrote: > We laboriously enforce parameter values are between one and some > arbitrary limit in length. Only RBD_MAX_IMAGE_NAME_SIZE comes from > librbd.h, and I'm not sure it applies. Where the other limits come > from is unclear. > > Drop the length checking. The limits librbd actually imposes must be > checked by librbd anyway. > > There's one minor complication: BDRVRBDState member name is a > fixed-size array. Depends on the length limit. Make it a pointer to > a dynamically allocated string. > > Signed-off-by: Markus Armbruster <arm...@redhat.com> > Reviewed-by: Eric Blake <ebl...@redhat.com> > --- > block/rbd.c | 91 > ++++++++++--------------------------------------------------- > 1 file changed, 14 insertions(+), 77 deletions(-)
Reviewed-by: Max Reitz <mre...@redhat.com>
signature.asc
Description: OpenPGP digital signature