nbd: Avoid magic number for NBD max name size
commit943cec86d0864464ab29b42940c49d1ccbe8d268
authorEric Blake <eblake@redhat.com>
Wed, 11 May 2016 22:39:44 +0000 (11 16:39 -0600)
committerPaolo Bonzini <pbonzini@redhat.com>
Thu, 16 Jun 2016 16:39:05 +0000 (16 18:39 +0200)
tree0edf2c73a1c3f3fee5a7f43cad329e5436618dcb
parentf3c32fce3688fe1f13ceb0777faa1fc19d66d1fc
nbd: Avoid magic number for NBD max name size

Declare a constant and use that when determining if an export
name fits within the constraints we are willing to support.

Note that upstream NBD recently documented that clients MUST
support export names of 256 bytes (not including trailing NUL),
and SHOULD support names up to 4096 bytes.  4096 is a bit big
(we would lose benefits of stack-allocation of a name array),
and we already have other limits in place (for example, qcow2
snapshot names are clamped around 1024).  So for now, just
stick to the required minimum, as that's easier to audit than
a full-scale support for larger names.

Signed-off-by: Eric Blake <eblake@redhat.com>
Message-Id: <1463006384-7734-12-git-send-email-eblake@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
include/block/nbd.h
nbd/client.c
nbd/server.c