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.

Test build failure mail thing

buildbot has a "send mail upon build failure" option. This should test that.

We'll revert the break once everything seems to work ;-)

Make the implementation of size_autodetect follow the documentation

The documentation claimed that size_autodetect could return UINT64_MAX
if size autodetection would fail. In fact, however, it would call err(),
which is for fatal errors.

Additionally, the final test (to use lseek SEEK_END) would never be
called, since it was preceded by an if structure which would call return
in one of its branches and err() in the other.

Remove off_t from what will eventually be a public API

Since the size of off_t may vary depending on whether the LFS defines
have been set, it is not a good idea to expose that, as the recently
fixed bug has shown. As long as we use nbdsrv.h internally only this
doesn't matter, but since the long-term goal is to make libnbdsrv a
public library, we'd better avoid such an issue.

We'll still use off_t internally, but that won't be exposed anymore.

Remove duplicate ISSERVER definition

Reorder LFS includes so it actually works

Add missing lfs.h include

lfs.h has an influence on sizeof(off_t), which has an influence on the
offsets of the SERVER data type, which makes things fail on 32 bit...
whoops.

Fix cleanup in simple_test

If nbd-server dies during the test for some reason or other, then the
kill call in the cleanup subroutine would fail, causing (due to 'set
-e') the script to abort immediately. As a result, the "rm -rf" of the
tempdir wouldn't happen, which would leave cruft in /tmp.

Fix.

Fully initialize hints

POSIX 1003.1 says:

 In this hints structure every member other than ai_flags, ai_family,
 ai_socktype, and ai_protocol shall be set to zero or a null pointer.

which Linux doesn't enforce, but FreeBSD does.

Fix missing include

Linux automatically includes <netinet/in.h> as a dependency of one of
the other include files, but this is not required by POSIX, and FreeBSD
doesn't do so.

Since we directly use some of the structs from that file,
add the include

Prepare for 3.6

Switch to xz (and away from bzip2)

Might change back, depending on discussion on mailinglist.

Fix build

If we introduce a new file, we need to make sure to add it to the
Makefile, too, otherwise it won't be distributed...

Add check for size_autodetect

Move size_autodetect to library

Add check for getmaskbyte(), too

Don't forget to distribute this new "macro.h", too

Add a check for append_serve, too

Separate out a useful macro

Update comment to reflect today's realities

Move "append_serve" to library, too.

That completes what can be moved without rewriting anything.

Whitespace fix

Fix Makefile.am's so that "make distcheck" works again

Add .gitignore with compiled files in this directory.

Make it compile (and pass)

Add test for dup_serve

Move dup_serve() to library

Enable maintainer-mode by default

Do a htonl conversion of opt.

Closes github issue #6, but breaks backwards compatibility on
little-endian systems.

I've decided not to care anymore about that. Cross-endian same-version
compatibility matters more than backwards compatibility.

So there.

Add a "code" test suite

This test suite will test functions in libnbdsrv.la. For now, only test
that the authorized_client function does what it's supposed to do.

Make a start with a library

The library does not contain much useful stuff yet, so make it a
noinst_LTLIBRARY for now, rather than a lib_LTLIBRARY.

As we move code to the library, we'll modify it so that it becomes
selfsufficient, and clean it up as we go.

For now, only move the freshly rewritten authorized_client() function
there.

This adds libtool to the list of stuff needed when compiling from git,
but that's not much of an issue.

Separate the test suite out into a separate directory

... in preparation of also adding a "code" test suite which tests
functions in the code, not runtime behaviour of nbd-server

Bring documentation in sync with code

Improve acl file parsing robustness

We now support comments, empty lines, and leading whitespace (which all get
skipped)

Lines with junk will _probably_ get skipped too, depending on how bad
the junk really is.

Unbreak virtstyle parameter parsing

There have been issues with the "cidrhash" option of the "virtstyle"
parameter since a long time, in that the way things were being parsed
wasn't very endian-independent.

Fix it the same way we just fixed the authorized_client function.

Fix authorized_client implementation a bit more

We previously added a fix for CVE-2013-6410, but that one accidentally
broke checks for non-netmasked entries entirely, rather than limiting it
to what it was actually supposed to allow. '\n' is also a character...

This fixes it properly, by not relying on string functions anymore, but
instead parsing all IP addresses to their binary representation, and
comparing that.

As an added bonus, we now also support IPv6 addresses in the ACL file.

Merge pull request #15 from alexsn/master

Fix potential socket leak in opennet()

gznbd: fix warning with newer zlib

When you build against newer zlib, you get a lot of warnings like so:

gznbd.c: In function 'main':
gznbd.c:87:5: warning: assignment from incompatible pointer type [enabled by default]
gznbd.c:109:5: warning: passing argument 1 of 'gzread' from incompatible pointer type [enabled by default]
In file included from gznbd.c:37:0:
/usr/include/zlib.h:1313:21: note: expected 'gzFile' but argument is of type 'struct gzFile_s **'
gznbd.c:118:9: warning: passing argument 1 of 'gzerror' from incompatible pointer type [enabled by default]

This is because the zlib API uses just gzFile everywhere, not a pointer
to a gzFile.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Wouter Verhelst <w@uter.be>

gznbd: use PRId64 for printing 64bit types

Gcc complains about printing these 64bit types:

gznbd.c:199:1: warning: format '%Ld' expects argument of type 'long long int',
but argument 5 has type 'u64' [-Wformat]

Use the standard defines from inttypes.h to avoid this issue.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Wouter Verhelst <w@uter.be>

Add the actual text of the GPLv2, as the license requires us to

Update nbd-client.c

Fix potential socket leak in opennet()

Prepare for 3.5

Use strcmp() rather than strncmp()

This results in some false positives. If the authfile contains:

192.168.0.12

and the client is

192.168.0.1

then access will be granted, because the strcmp was limiting to
"192.168.0.1" for no particularly good reason.

We should also canonicalize our names and work on that, rather than
doing a simple strcmp(), but that's for later.

Fix documentation to reflect reality

3.4 was released...

re-enable integrityhuge test

We disabled it at the time because it would deadlock in some cases
(though not on my machine).

Since then, some deadlock fixes have accumulated. I don't know whether
the culprit has been found and remedied, but it doesn't hurt to try. If
it turns out to still be a problem, we can always disable them again
afterwards.

While here, remove generated files ("missing") from git.

Remove the same ioctl() call in another place

Not sure why we do this twice anymore, but hey.

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

Remove obsolete NBD_CLEAR_QUE ioctl.

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

nbd-server: const'd function parameter variable

Making the parameter variable const revealed a smelly assignment
statement which was deemed unnecessary and hence removed.

Signed-off-by: Tuomas Räsänen <tuomasjjrasanen@tjjr.fi>
Signed-off-by: Wouter Verhelst <w@uter.be>

nbd-server: remove superfluous 2nd parameter from handle_oldstyle_connection()

A pointer to the server struct is passed anyways, the socket can be
accessed via that pointer.

Signed-off-by: Tuomas Räsänen <tuomasjjrasanen@tjjr.fi>
Signed-off-by: Wouter Verhelst <w@uter.be>

nbd-server: refactor out common code

Signed-off-by: Tuomas Räsänen <tuomasjjrasanen@tjjr.fi>
Signed-off-by: Wouter Verhelst <w@uter.be>

nbd-server: rename handle_connection() -> handle_oldstyle_connection()

The new name describes better the actual usage.

Remove also the fourth parameter, CLIENT*, which was always NULL.

Signed-off-by: Tuomas Räsänen <tuomasjjrasanen@tjjr.fi>

nbd-server: handle modern-style negotiation in a child process

Previously, the modern style negotiation was carried out in the root
server (listener) process before forking the actual client handler. This
made it possible for a malfunctioning or evil client to terminate the
root process simply by querying a non-existent export or aborting in the
middle of the negotation process (caused SIGPIPE in the server).

This commit moves the negotiation process to the child to keep the root
process up and running no matter what happens during the negotiation.

See http://sourceforge.net/mailarchive/message.php?msg_id=30410146

Signed-off-by: Tuomas Räsänen <tuomasjjrasanen@tjjr.fi>

Clarify that we're using tests which expect the serial test protocol

Automake changed its defaults from using the serial protocol to using
the parallel protocol sometime between 1.11 and 1.13.

Our test suite makes a few assumptions which are valid for the serial
protocol, but not for the parallel protocol, which makes the test suite
fail with 1.13. As such, explicitly state that we want the serial
protocol so that things don't break with recent versions of automake.

A proper fix would be to make the test suite work with the parallel
protocol, but that's a lot more work than we have time for right now.

nbd-server: narrow the scope of a function-local variable

It's better to keep variables strictly inside their usage scope than in
more wider scope to avoid unintentional variable shadowing and leaking.

Signed-off-by: Tuomas Räsänen <tuomasjjrasanen@tjjr.fi>