Add write test to test suite

Remove compilation warnings and increase error checking in test suite

fix autogen.sh

Building from source fails on a clean git pull, due to autogen.sh failing.

$ ./autogen.sh
+ make -C man -f Makefile.am infiles
make: Entering directory `/home/amb/nbd/nbd/man'
make: *** No rule to make target `infiles'. Stop.
make: Leaving directory `/home/amb/nbd/nbd/man'

A manual autoreconf does this.

$ autoreconf -f -i -s
configure.ac:123: required file `man/nbd-client.8.sh.in' not found
configure.ac:123: required file `man/nbd-server.5.sh.in' not found
configure.ac:123: required file `man/nbd-server.1.sh.in' not found
autoreconf: automake failed with exit status: 1

I think the attached is the right patch (well, I have ./configure
now). There may be a prettier way of doing it (e.g. putting an
"infiles" target into man/Makefile.am) but this is least intrusive.

--
Alex Bligh

Signed-off-by: Alex Bligh <alex@alex.org.uk>

Ensure all options have option length

This is a documentation only patch to the new handshake protocol, but
changes the protocol in completely back compatible manner.

Under the current handshake protocol, it is impliedly necessary for the
server to understand every option presented by the client, as there is
no way for the server to skip options it does not understand (as the
length field is not always part of the option).

My proposal is to include the length field as a 32 bit unsigned length
in the same way that it currently forms the first 4 octets of the
NBD_OPT_EXPORT_NAME. So the length field has technically moved out
of the option data and into the option header (obviously there is
no code change required). This will allow servers to skip options
presented by clients that the server does not understand.

--
Alex Bligh

Signed-off-by: Alex Bligh <alex@alex.org.uk>

Release 2.9.21a

Add sh.tmpl to EXTRA_DIST so it won't break

Release 2.9.21

Fix 'make distcheck'

Fix thinko

It's nbd-client.8, not nbd-server.8

Don't forget the template, and ignore generated files

Manpage generation revisited

Previously, we would generate manpages using direct automake-style @foo@
substitution. These, however, contain data in shell syntaxis, which
results in a manpage in the style of '${prefix}/etc/nbd-server/config
allows to configure the nbd-server', which is confusing.

Instead, generate a shell script that outputs the manpage using shell
interpolation, which supposedly works correctly. It's a bit yuck, but
it's the way automake wants us to do these things.

Documentation updates

- Document the protocol
- Move developer documentation into a separate directory. This includes
  the Doxyfile. Update some other files so that doc/ isn't git ignore'd,
  and so that the Doxyfile is generated at the right location.
- Add a README which explains the purpose of that directory.

Fix dup_serve

Copy the virtstyle field too, and make sure that we copy everything from
's' (the source) rather than from 'serve' (the destination)

Fix memleak

Fix inline usage

inlines must be static, or gcc with some particular set of options (C99?
-Wall -Werror? dunno, don't care) won't compile it.

Require C99

We'll want to use a few C99-specific features, and C99 is getting older
than whisky now anyway.

Fix getopt usage

getopt returns an int, not a char. Since char is unsigned by default on
some architectures, this caused an infinite loop in option parsing
there.

Check return value of read_all in all cases

... to avoid nbd-tester-client going berzerk if the server dies (this is a test
suite! It should deal with servers dying!)

Assign a value to serve when using modern protocol

We dereference this pointer later on, so need to make sure it's
actually valid.

As an added precaution, initialize it to NULL, so that we catch any
further incorrect dereferences, too.

Release 2.9.20

Test request blocks that are larger than 1MB, too.

Fix handling of oversized requests

Previously, nbd-server would not serve requests that were over 1MB in
size. This was a problem, and even a bug since the kernel does send such
requests.

Modify the server so that it loops if the request is larger than 1MB.

Fix buffer size checking

Yes, this means we've re-introduced CVE-2005-3534. Sigh.

Remove outdated roadmap

Release 2.9.19

10 TByte partition

Moin!

The attached patch allows to run a 16TB nbd partition on a 32-bit
system.

Gruß, Stephan

nbd-server: Limit the number of connections

Hi there,

This is a patch that allows to limit the number of opened connections in
nbd-server.

This limitation can be specified from both command line and config
files:
- -M <n> or --max-connections <n> from command line
- or maxconnections = <n> in config files (export sections)

Regards,
Greg

Put nbd-client back in sbin

Move man files to man/

Signed-off-by: Laurent Vivier <laurent@vivier.eu>

Revisit documentation

There were a few issues with the documentation, where it grew some hairy
and weird phrasing. Fix those.

Remove GCC warnings:

warning: ignoring return value of ‘write’
warning: ignoring return value of ‘read’
warning: format not a string literal and no format arguments
warning: ignoring return value of ‘daemon’

Signed-off-by: Laurent Vivier <laurent@vivier.eu>

Be less scary

When nothing is configured, a rather scary warning is displayed. Change
it so that it isn't quite as scary, since that produces a number of
questions from users who don't know nbd-server.

Oops

README: nbX -> nbdX

include sdp_inet.h for build with SDP

fix build with non-gcc compiler

Pulled from Alt Linu

Conflicts:

cliserv.h

Release 2.9.18

Extend test suite and fix named exports

Comment out <linux/types.h> include

We provide the necessary bits ourselves anyway, and it doesn't exist on
non-Linux systems.

Use a recent enough version of nbd.h (oops)

Release 2.9.17

Document differences with old style exports

Fix simple_test for named exports

Old-style exports require the oldstyle=true option in the 'generic'
section; so add that.

Prepare simple_test for support for named exports in nbd-tester-client,
too, but that still needs to be implemented.

Fix command-line oldstyle export

Use IANA-assigned port number: 10809

Sanitize negotiation a bit

- We don't need 64 bit size_t for a name
- We don't need 64 reserved bits client to server; if we ever need more
  than 31, we'll just add another field and use the last bit as a marker
  to say that the other field is available.

Documentation updates

- Document -N / -name option
- Some generic docbook fixes

Merge branch 'master' into names

Mention two-dash form of long options

Return the right value on nbd-client -c

command line parsing rewrite accidentally removed this.

Fix a host/network order bug

Functional fixes

Result of tests. It works! Whee!

Documentation update

Actually disable oldstyle export when requested

Use a different magic

Use a different magic, to avoid old clients accidentally using the
well-known port and hanging.

Initial implementation of named exports

Fix segfault

Documentation update

Prepare for 2.9.16

Make -Wall clean

Add nbd.h to source

Previously, we would depend on an out-of-tree nbd.h. This is error-prone
on Linux (if the kernel nbd.h changes incompatibly, the userspace tools
won't compile anymore) and problematic on non-Linux (where you'd have to
manage somehow to fetch a working nbd.h)

So, stop being silly.

Generate nbd-client.8.in, too

Whitespace fixes

Remove cruft

This was useful when we tried making github our official distribution
point; but we're not doing that anymore, so drop it.

Improve structure

- Use getopt_long_only() to implement option parsing. This probably
  makes use of small C libraries harder, but that wasn't very much
  possible to begin with, anyway.
- Deprecate bs= and timeout= options in favour of new --block-size and
  --timeout options.
- Compartmentalize a bit more. Yes, the compiler will optimize that away
  in some cases, but what do I care, I don't need to write assembler
  code.

Documentation update to follow.

Fix compile with -DNOFORK

Fix logic reversal in authorization file

Deprecate command line export options.

Whitespace fix

Use correct names

The kernel uses /dev/nbdX rather than /dev/nbX. MAKEDEV used to be
different at some point, but no more.

Fix bugs in command-line argument handling

We produced an error message when all four flags were used; and we would
only catch such flags if they were produced in the right order (which
was different from the order in the manpage).

Thanks to Fan Zhang for catching these.

Fix leaks

There were a few strings that weren't being removed. This is
cleanup-code that's not called all that often, but We Shouldn't
Leak(TM).

Improve output

- When using a float to store a result, make sure we don't do integer
  calculation.
- Add our PID to the output, so that we can understand which line of
  output is part of which nbd-tester-client run.

Missed a bit

Documentation update

Since we now support IPv6, upodate those bits of the documentation that
assume v4 only.

Release 2.9.15

Bug#382189: IPv6 support

Package: nbd
Version: 2.9.14-3
Severity: normal

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

The latest patch to master branch of git repository is attached.
Please review the patch again.

It's also solved the #557810.

Add append_serve() more over the last patch to verify the parsing
listening address which now the parsing listening address could
be IPv4/IPv6 and also hostname. The service will resolve the hostname
and try to listening on all of them.

If the hostname/address could not solve, must show the error

getaddrinfo failed on listen host/address: testing (Name or service not known)

If parsing hostname,

It will listen on 127.0.0.1:12345

It will listen on ::1@12345

Regards,
Neutron Soutmun

- -- System Information:
Debian Release: squeeze/sid
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-2-amd64 (SMP w/2 CPU cores)
Locale: LANG=th_TH.UTF-8, LC_CTYPE=th_TH.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)

iEYEARECAAYFAkupomIACgkQ1k7Ar9TO/TdLngCgvGp2oGKR0PLGmMFVTUkxh9yO
NrQAnR4FqtwoEuo/aZoEFhdhFI3mOVbU
=1UBe
-----END PGP SIGNATURE-----

Be explicit when we exit

Currently, when a fatal error occurs, nbd-server logs that error, but
then silently exits -- i.e., it is left to the user to guess that it has
exited. We should be more explicit about that fact.

Parse listenaddr for hostnames

Previously, a listen address had to be specified by IP address;
hostnames where not supported. This patch should fix that. It is useful
for those who wish to use something like 'listenaddr = localhost',
though using the IP address is really recommended for most other
situations.

Fix 'make -j check'

Port numbers were the same across all three tests, so that made the
check fail if they were ran at the same time...

lfs.h: fix build with --disable-lfs

Running ./configure with --disable-lfs #define's NBD_LFS to 0 and
doesn't #undef it, so the LFS check should use #if rather than #ifdef
similar to how it is done in cliserv.h

Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>

Prepare for 2.9.14

Make -Wall -Werror compile again

Document why this is necessary.

Put the -nofork check on the right spot

nbd-client will fork in two occasions: once when going daemon, once when
forking itself off to be able to open the device so we can trigger the
kernel to reread its partition table (if any), which in turn will cause
udev to DTRT.

The NOFORK macros were written in to make debugging simpeler, and
predate the daemon() thing. I think. Anyhow, what's useful for code
debugging is not necessarily useful for runtime debugging; so this moves
the daemon() call inside the macros, makes it not be called if -nofork
has been specified, and changes the code so that the partition table
reread is still triggered even if -nofork has been specified.

Missed one

This one, too, needed to be thrown out. Oops.

Remove unsupported options

The no longer supported -a option was still part of the 'getopt' call,
and was still shown in the error output. This patch removes that code.

Make autogen.sh work

Makefile.am expects $(srcdir) to be set -- so set it in the environment

Remove dead code

If someone needs it, they can always pull it from git.

Also, make the mlockall() call conditional on if(swap) again.

Documentation update

Sometimes users get confused over the fact that block devices are files,
too, for the most part. Make it a bit more explicit that exporting block
devices is supported in the documentation.

Fix error output

dousers() previously tried to call g_message directly. Since we're
running as daemon already at this point, g_message won't produce output.

Fix by using err() instead.

Fix message string

Adding the filename as an argument is nice, but the message string needs
it too

Patches from Debian

Rerun autofoo

Make incorrect use of timeout fail.

When nbd-client is compiled on a system that does not support the timeout
ioctl(), we now fail with an error message. Previously, the option would be
silently ignored, which might not be optimal.

Unconfuse

Implement sync option

This allows a user to enable fsync() calls after a write to a backend
file.

Prepare for 2.9.13

Documentation update

People are sometimes confused about the warnings that nbd-server
produces when an invalid or incomplete warning is encountered.

Note in the manpage that these warnings are harmless if a command-line
configuration was specified and no configuration file is desired.

Don't try working on partitions

When a partition is given to the -d or -c command line options, reduce
it to the master device instead.

This assumes udev-style naming of device entries, though.