server: Avoid -Wshadow warnings
None of these shadowing instances appear to misbehave, but it's
confusing to tell if code intentionally meant for a tighter scope to
override a broader one. Affected:
global 'readonly' renamed to 'read_only' given the number of other
places that had a local parameter 'readonly'
in main(), 'p' was declared twice with no overlap in use, but 'q'
once; move 'q' to the first declaration
in main(), 'filename' and 'short_name' were declared twice with no
overlap in use; use the first instance, and change the type of
short_name to bool to match is_short_name()
in make_random_fifo(), the local 'unixshadow' is renamed to 'sock' to
no longer shadow the global 'unixshadow'
all callers of send_newstyle_option_reply_exportname() passed the
global 'exportname', no need for a local parameter 'exportname'
in handle_request(), 'buf' for NBD_CMD_CACHE renamed to 'dropbuf' to
be distinct from the function parameter 'buf'
Signed-off-by: Eric Blake <eblake@redhat.com>