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.

Fix autodetection

Everything else in nbd-server was already 64bit-clean, but this part,
where the size of a block device is autodected, wasn't.

Replace the ioctl() to BLKGETSIZE with its 64bit version, so that trying
to detect the size of a multi-terabyte block device does not fail.

Avoid unaligned access

nbd-tester-client tried to write to a char array in a packed struct by
casting it to an u64 pointer and dereferencing it. That causes unaligned
access and, hence, a SIGBUS on SPARC.

Fix by using memcpy instead.

Actually go daemon

just doing chdir("/") leave file descriptors open, which is not ideal.
Doing it this way closes everything, including any system-specific
things we're not aware of.

Apparently forgot this file -- update that one, too.

Merge branch 'import'

Update documentation to reflect new behaviour of -swap option

Document -noswap option

Hacked nbd-client.c to include a flag -nofork, that allows the nbd-client process to not fork and daemonize itself.

Signed-off-by: Wouter Verhelst <w@uter.be>

Fix for nbd-server uner xinetd

nbd-server currently fails under inetd due to GLib error messages being
reported over the data channel. The attached patch moves the configuration
file parsing warning to occur after the inetd special-case check and
redirects all GLib messages to the syslog (if --enable-syslog is used).

Remove support for timeout option

The timeout option has been broken for a very long time. Additionally,
something like this is better done by making use of TCP keepalive
probes, which nbd-server has implemented since 2003 (unless you're using
it in inetd mode, in which case you should add the keepalive option to
the inetd configuration for the nbd-server).

more user-friendly error message

Hi folks,

we recently had a bug filed against the nbd Fedora package claiming
about the clarity of the error message which is printed when the nbd
module is not loaded:

https://bugzilla.redhat.com/show_bug.cgi?id=496751

What about merging a tiny patch I attached?

Regards,
Milos

autoconf after version...

version number

rerun autotools

More build fixes

these let 'make distcheck' work again.

Fix build system

* Add lfs.h to the proper _SOURCES entry
* remove winnbd from SUBDIRS, since we don't have that anymore.

use atoll instead of atol

... so that multi-terabyte exports work, too.

Minor fixes

- Gracefully handle when a config file contains a valid [generic] section, but
  no export sections
- Free our key file stuff on successful parsing of the config file, too.

Avoid i18n when we don't need it

Manpages aren't translated, so translated dates there are sucky.

Change meaning of -swap option

Change meaning of -swap option: rather than trying to use a kernel patch
which has been effectively dead since sometime around 2.1.something, do
an mlockall() so that the kernel won't throw us out.

The MCL_CURRENT | MCL_FUTURE bit might prove to be problematic, but if
so, we'll deal with it when we get there.

Build system fixes and documentation

Should make compiling on non-Linux operating systems not quite as painful