Drop support for the oldstyle protocol

The newstyle protocol was originally written for NBD 2.9.17, released in
2009. We are 2015 now. People have had over five years to migrate their
setups now.

Additionally, due to the fact that I originally wrote support for the
newstyle protocol in a somewhat confusing way, maintaining that part of
the code has become harder than it needs be. A bug did now appear
because I couldn't read my own code anymore.

While it should, in theory, be possible to just fix the damn code while
still retaining support for the oldstyle protocol, it's easier (and not
too unreasonable) to just drop that instead.

Also fix the test suite so it doesn't try to run tests using the oldstyle
protocol anymore

Prepare 3.9.1

remember in global variable whether or not to send zeros

For modern connection function negotiate() is called twice, but client
flags are sent only once. So remember in global variable whether or not
to send zeros.

Signed-off-by: Christian Hesse <mail@eworm.de>

Release 3.9

Make test suite script be more helpful when run in "set up config" mode

Don't forget to signal success when things work

Extend test suite to test AF_UNIX functionality

Document unix domain server mode

Add server-side support for UNIX domain sockets

Don't forget to close the DIR* when we're not in error handling

Reorder options

GNU getopt() will happily reorder non-option arguments if we ask it to
(which we do), but that's a GNU extension. Since we want to support
FreeBSD too, don't put these arguments at the end, but put them before
non-option arguments.

Whoops; fix missed logging() call

Unbreak --enable-syslog

Merge CorvusCorax/treefiles

Adds support for exporting a single export as a swathload of files. This
is similar to, but not exactly the same as, the multifile export mode.
The difference lies in the fact that multifile expects the files to be
of differing sizes, and will not remove them; in contrast, the treefiles
mode will expect them all to be a single page in size, and will remove
them on a TRIM command.

This mode is useful for FUSE-backed exports which may have their storage
on a remote server (e.g., SSHFS), where it would be inefficient to
transfer the entire export for every (small) write.

* CorvusCorax/treefiles:
  added test cases for tree mode
  got rid of CamelCase
  modified documentation to reflect removal of -t command line switch
  Changed treefile implementation for better code compatibility
  removed "treefiles" option from commandline as commandline is deprecated by policy and no new features must be added
  removed new documentation section which had accidently been added twice
  fixed error handling, messages, cleanups
  fixed typo
  Fix mistake in trim address calculation
  Added support for "treefile" representation with one file per block in a subdir tree structure

Make -F work on FreeBSD, too

added test cases for tree mode

got rid of CamelCase

modified documentation to reflect removal of -t command line switch

Changed treefile implementation for better code compatibility

removed "treefiles" option from commandline as commandline is deprecated by policy and no new features must be added

removed new documentation section which had accidently been added twice

fixed error handling, messages, cleanups

fixed typo

Fix mistake in trim address calculation

Merge branch 'master' into corvuscorax/treefiles

Conflicts:
nbd-server.c

Added support for "treefile" representation with one file per block in a subdir tree structure

Merge branch 'master' of github.com:yoe/nbd

* 'master' of github.com:yoe/nbd:
  Make this listen to localhost only, too
  Add missing include

Add license statement

Acked-By: Paolo Bonzini <pbonzini@redhat.com>
Acked-By: Alex Bligh <alex@alex.org.uk>

Merge branch 'master' of github.com:yoe/nbd

Make this listen to localhost only, too

Add missing include

setsockopt() wants <sys/types.h> and <sys/socket.h>. Add.

Add missing return

CID#1193182

Fix out-of-bounds write

In the (unlikely, if not impossible) case that that read returns 256
characters, we'll still be overwriting one stack byte, which is Wrong(tm).

CID#1193183

Merge https://github.com/bamos/nbd

* https://github.com/bamos/nbd:
  Disable swap option on Android.

Disable swap option on Android.

Hi, this is a followup on http://sourceforge.net/p/nbd/mailman/message/32862671/.

I've added options to disable the swap option when compiling for
with android-18, arm-linux-androideabi-4.8, and the Android NDK r10b.
I'm using the below snippet to create a standalone toolchain
and compile, and I'll try to keep my post at
http://bamos.github.io/2014/09/08/nbd-android/ synchronized
with the master branch of nbd, so feel free tag me if
any changes are made to this.

Regards,
Brandon.

---

die() { echo $*; return -1; }
touch man/nbd-{client.8,server.{1,5},trdump.1}.sh.in
autoreconf -f -i || die "autoreconf failed."

$ANDROID_NDK/build/tools/make-standalone-toolchain.sh \
  --platform=android-18 \
  --toolchain=arm-linux-androideabi-4.8 \
  --install-dir=$PWD/toolchain

pathadd() { PATH="${PATH:+"$PATH:"}$1"; }
pathadd "$PWD/toolchain/bin"
command -v arm-linux-androideabi-gcc &> /dev/null || exit -1
export SYSROOT=$PWD/toolchain/sysroot
export CC="arm-linux-androideabi-gcc --sysroot=$SYSROOT"
export CXX=arm-linux-androideabi-g++
export LD=arm-linux-androideabi-ld
export RANLIB=arm-linux-androideabi-ranlib
export AR=arm-linux-androideabi-ar
export CROSS_PREFIX=arm-linux-androideabi-
export CFLAGS='-march=armv7-a -mfloat-abi=softfp -mfpu=neon'
export LDFLAGS='-Wl,--fix-cortex-a8'

./configure --host=armv7-unknown-linux --target=armv7-unknown-linux \
  || die "configure failed"
make nbd-client

Update name.

Update message number, after consultation with Takahiro

Fix uninitialized variable in oldstyle negotiation

Coverity CID#1248168

Document existing request types in a somewhat more detailed manner

Modernize README, and point to sourceforge download pages for the benefit of github users.

Point to mailinglist

Extract the unix socket code from opennet() into a distinct openunix().

Fixed previous mistakes with unix socket code.

Adding capability for the client to connect to a unix socket.

Avoid sending 124 zeroes of useless

Handle absense of (struct dirent).d_type

*BSD and Linux implement this, but it's not required by POSIX and, e.g.,
Solaris/IllumOS don't support it.

Since we already have to deal with DT_UNKNOWN and call the stat stuff in
that case anyway, just do some autoconf and preprocessor magic to hide
the differences.

Should fix github.com issue#22.

Clarify that gznbd is slightly outdated

Merge https://github.com/ahippo/nbd

Don't use XFAIL_TESTS, instead make nbd-tester-client exit with appropriate error state

Test that trying to write to a read-only export fails

Document a way for a default export

gznbd: use atoll() to convert size argument

gznbd works incorrectly with large size argument which overflows 32-bit integer.

For example:
gznbd /dev/nbd0 /root/test.gz 1000204886016
gznbd: 1000204886016 does not appear to be a valid size

The size is stored in u64, but is converted from a string using atol().
Simply replace atol() with atoll().
gznbd is compiled with -std=gnu99 (as the whole nbd), so atoll() is available.

Links to the original patch:
[1] https://sourceforge.net/p/nbd/mailman/nbd-general/thread/560411311771602%40web126.yandex.ru/#msg27860052
[2] http://marc.info/?l=nbd-general&m=131177162001072&w=2

Wire gznbd into general configure.ac/Makefile.am

Add --enable-gznbd configure option (disabled by default).
Avoid multiple recompilations of cliserv.c by factoring it into a static
library.

This fixes gznbd compilation broken after
43b96ec0d22c0a42c20104cb66aa2d2236da3e77:

gznbd.c:(.text.startup+0x420): undefined reference to `ntohll'

Make short version of -V actually work

Document -V option

Fix various warnings that pop up by default with gcc 4.9

Replace return by break statement

Both are extraneous, really, but keep the break there for consistency (and to
avoid "eep, forgot... ah, no" kind of things)

Using return instead (without argument) makes clang choke, which is useless.

Update nbd-server.c

Added a parameter to output a version string.

Revert "Enable automatic coverity coverage"

This reverts commit eb6365af2c4c8eecf62317940c06f2bc58810a98.

The travis stuff isn't supposed to work that way; also, we're doing
weekly coverity builds with buildbot now, so it's less relevant.

Implement new config file option "cowdir"

Allows to specify a location where copy-on-write files should be
written.

Update clientacl.c

Update test code with new prototype of address_matches()

Pass whole struct sockaddr to function address_matches() and use it to
compare against the masks defined in auth file.

Fix obi-wan (whoops)

add missing include syslog.h

Clear ai on return, also in case of error

Coverity CID#1238644

Ensure pidftemplate is always 0-terminated

Coverity CID#1238650

Ensure the "errstr" buffer is always 0-terminated

Coverity CID#1238651 (and many others)

Don't try to be smarter than the compiler about inlining

When we specify "inline int foo()" (as opposed to "static inline", I
guess), compilation may fail when compiling with -O0, as I sometimes do
to simplify debugging.

The compiler is way smarter about deciding when things are best inlined,
anyway. So stop trying to tell it what to do.

Don't warn about oldstyle exports if oldstyle is set to true.

Bug reported through Ubuntu Launchpad

Fix up parse_cfile signature

parse_cfile repurposed the genconf parameter as a boolean flag (in
addition to its storing of generic configuration) to signal the
expectation (or absense thereof) of a [generic] section in the
configuration file.

Not only is this confusing, it also makes it impossible to reliably
detect whether we want an oldstyle export. Fix for that in the next
commit.

fix typo

do not try to open authorization file without filename

Guard against errors

Enable automatic coverity coverage

Make the travis stuff work (after some testing on a separate branch)

Clean up cliserv.h

Previously, cliserv.h also contained some functions, which would then be
defined twice if we hadn't inlined them.

That is a fairly ugly way of doing things, so instead move the few
functions in cliserv.h to a new cliserv.c, and compile and link that in.

This should also make more recent versions of clang not be unhappy about
"missing" functions

Install a more recent version of automake through apt pinning. I hope.

Figure out which versions of automake we have to work with

autotools need to be regenerated

Add travis config file

Prepare for 3.8

Don't go out of bounds in an array...

Zero-terminate buffers with data supplied by remote parties

Found by coverity scan

Close the directory handle when we no longer need it

Found by coverity scan

Add assertion, to make LLVM static analyzer happy

Revert "Make this part of the code at least look similar"

This reverts commit 701af5d057f523c714517085469dec556aa76b28.

The test suite apparently expects this to be there... do this properly
later on.

Support systemd initrd kill protection

systemd wants us to mark our argv[0] when we don't want to be killed.

Since I think that's a bit ugly, add an option so it's only done when
necessary.

Make this part of the code at least look similar

First we assert that s != NULL.

Then we check if a != NULL, and return -1 if it is. That just makes no sense.

Change the check for a to also be an assertion, and make things at least look similar.

Silence warning

Be less verbose

If we list each and every request, that means a build log becomes huge
and unwieldy, so reduce the chatter somewhat.

Add build status applet, too

Somewhat ugly, but it works -- and that's what matters most.

Disable integrityhuge test

nbd-server currently may deadlock on huge reads or writes, especially on
slower machines.

We're fixing this on the io_transaction branch, but meanwhile disable
integrityhuge again.

Release 3.7

Add README.md to Makefile.am

README files are distributed automatically, but README.md files are not;
so add it explicitly

Create Coverity model

Fix typo

the "dir" parameter contains the name of the directory, while the "dirh"
variable is an opendir() handle that'll only be NULL in case of error.

By complete accident, this compiled, but it wouldn't actually ever do
anything useful.

Don't leak resources

Close the authorized file in _all_ cases, not just when authorized.

Found by coverity scan.

Reformat README to markdown

Make things work again

Tickle the git repository

Did something wrong, hopefully it will work now.