Merge branch 'refs/heads/t/fchdir-warning-fix'

Was cooking for nine months...

* refs/heads/t/fchdir-warning-fix:
  fd.c: Avoid gcc warning regarding fchdir().

Merge branch 'refs/heads/t/dbdir-fix'

The topic branch started on 2016-06-02 and was cooking since
2016-06-24. It contains a single fix.

* refs/heads/t/dbdir-fix:
  Abort if neither --database-dir nor --base-dir is given.

create: Make the directory scan more robust.

The directory traversal of adu calls adu_opendir() of fd.c for every
subdirectory. If this call fails with EACCESS, a warning is printed
but the operation continues, ignoring the directory which could not
be accessed. Other errors are considered fatal, though, and adu aborts
in this case.

This patch continues to scan in this case. This is achieved by making
all errors from adu_opendir non-fatal. In the error case we print a
warning and ignore the directory, but no longer abort.

Thanks to Steffen Schmidt for pointing out this flaw.

Abort if neither --database-dir nor --base-dir is given.

If --database-dir is not given, adu concatenates the database root
directory (which defaults to /var/lib/adu) with the value of the
--base-dir argument. However, if --base-dir is not given either,
base_dir_arg is NULL, and the concatenated string becomes the result
is /var/lib/adu(null), at least on glibc.

With this patch applied, adu handles this case more gracefully: it
prints an error message and aborts early.

build: Don't remove adu.png on make clean.

This was appropriate with the old logo which was generated from a
.sk source file. The new logo, however, only exists in png format,
so we should never remove it.

The adu.sk~ file can also be skipped from the list of files to be
removed by make clean.

fd.c: Avoid gcc warning regarding fchdir().

gcc-4.8.4 on Ubuntu-14.04 complains about our use of fchdir():

fd.c:181:9: warning: ignoring return value of ‘fchdir’, declared with attribute warn_unused_result [-Wunused-result]
   fchdir(*cwd);

Silence the warning by introducing a dummy variable. The patch also
fixes the indentation of the comment which explains why we ignore
the return value here.

adu-1.0.0.

This project is done. In fact it was done already five years ago
and has seen only a couple of minor fixes since then. With the new
version number it does not look like the project is dead.

Fix gitweb link.

Although on the gitweb server there is a link adu -> adu.git, the
link on the main page does not work without the .git suffix. This
used to work at some point...

fd.c: EAGAIN can not happen in __write().

The only caller does not specify O_NONBLOCK, so the check for EAGAIN
is unnecessary.

fd.c: Make __write() static.

It's an internal function which is only used inside fd.c, as indicated
by the leading underscores. And it does not have a prototype in fd.h.

Simplify and rename mmap_full_file().

There is only one caller which maps the file privately into read-only
memory. Moreover, this caller does not care about the file descriptor
from which the mapping was created.

This commit removes the argument from mmap_full_file() which control
the type of the map and the result pointer for the file descriptor,
thereby simplifying the function slightly. Rename it to mmap_file_ro()
to indicate that read-only private mappings are implied.

Make fd.c and select.c include their own header file.

This helps to spot bugs where declaration and implementation
differ. Fortunately, there is so such bug in adu, but it's a good thing
(TM) to let the compiler error out in this case.

Fix spelling of member of struct num_format.

Assorted typo fixes in comments.

string.c: Kill pointless initialization.

Found by cppcheck.

interactive.c: Kill pointless initialization.

Found by cppcheck.

Remove stale sentence about the logo.

The new logo was created with gimp, but this does not really matter,
so remove the sentence from INSTALL.

Replace systemlinux.org URL and email address.

Although the web page is still online at the old location, the new
location at tuebingen.mpg.de is now the "official" one.

This commit changes the URL in the INSTALL file and all email addresses
to the new domain.

Add new logo.

To paraphrase a similar change in the ratpoison project: it was hard
to say good-bye to the old logo but sometimes we just have to let the
past be the past.

Move adu_strerror() to adu.c.

This function is a bit too large to be inlined, and it depends on osl.

Remove VSPRINTF.

This macro has only a single user in string.c, and it's kind of
pointless for this code to be a macro.

Move adu_isspace() to interactive.c.

This macro is only used there, so it need not be public.

adu.ggo: Remove unused --config-file option.

This never worked since we do not even call the generated config file
parser. Also remove --conf-parser from gengetopt options.

interactive.c: Return proper error on icom_set() failure.

If an interactive set command fails, we currently reset the
configuration to the default and return success. This is probably
not what the user expected.

Make the command fail instead.

Print directory name on create errors.

In create mode, if the database directory already exists, adu prints
an error message of the form

main: directory exists.

This is a bit sparse, so let's add a log message which prints the
directory name. With the patch applied the output becomes

open_dir_table: could not create /ebio/maan/tmp/adu-lists/
main: directory exists

Fix user summary accounting if pattern is given.

In user summary select mode we iterate over each row in the user
info table and get the directory name from the number stored in the
row. Next we match this name against the given pattern and only take
matching directories into account for the user summary.

Currently we call get_dir_name_of_row(), but this is wrong:
Since we iterate over the user info table, we must call
get_dir_name_of_user_row() instead as the former function expects a
pointer to a dir table row.

This bug caused wrong directory names to be matched against the
pattern, hence the results of the user summary were incorrect.

Avoid gcc warning.

Introduce a default case for the body of the switch statement in
print_statistics() to avoid the following gcc-4.8.1 warning:

select.c: In function 'print_statistics':
select.c:806:2: warning: enumeration value 'select_mode__NULL' not handled in switch [-Wswitch]
  switch (select_conf.select_mode_arg) {
  ^

format.c: Remove an unused variable.

In format_items(), "unit" is set but unused. gcc is right to complain
here. This variable is pointless and can go away.

split_args(): Do not insist on checking the return value.

Currently, split_args() of string.c has the __must_check attribute
which instructs gcc to warn whenever the return value of this function
is ignored by the caller.

However, since the returned array is NULL terminated anyway, there
are situations the return value my safely be ignored, for example if
the returned array is passed to execvp(), which does not receive a
length argument.

This patch removes the __must_check attribute and fixes a "set but
not used" warning on newer gcc versions.

Fix check for return value of catch_signal().

Commit 2d7a4d61 made adu's signal handling portable by switching from
signal() to sigaction() for installing signal handlers.  This commit
added the new function catch_signal() which returns the return value
of the underlying call to sigaction(), i.e. zero on success, and -1
on errors.

However, embarrassingly enough, one caller of catch_signal() still
checked this return value against SIG_ERR, which is the value that
is returned from signal() on errors.

Fix this bug by testing the return value against zero.

Makefile: Honor CPPFLAGS also when creating dependencies.

Makefile.deps is created using a cc -MM -MG command which might need
the CPPFLAGS provided by the user, just as the ordinary cc commands
for creating an object file.

For example, the user might have passed -I$HOME/include, since
libosl was installed in $HOME. In this case we need to pass
this to the cc command that creates the Makefile as well.

fd.c: Make __chdir() static.

This function is only used in fd.c.

Fix signal handling.

Using signal() to set the disposition of a signal is always a bad idea
as POSIX does not specify whether a system call which was interrupted
should be restarted or not.

For interactive sessions, the Linux behaviour is to automatically
restart slow system calls, specifically read(2) in case nothing had
been read before the interrupt arrived. This is rather unfortunate
as adu calls fgets(3) (hence read(2)) in an endless loop to read the
user input. Therefore automatically restarted read() calls result
in interactive sessions that can not be terminated easily, as was
noticed by Sebastian Stark.

This patch makes the signal initialization code call sigaction()
instead of signal() to set up the handlers. This buys us well-defined
semantics across all operating systems, namely to *not* restart slow
system calls. As a side effect of this change, interactive sessions
can now be terminated by sending SIGINT (e.g., by pressing CTRL+C).

Minor help text improvements.

Fix typo in comment.

adu-0.1.0.

No bug reports and no commits for more than one year. This can only mean
that adu is ready for production use now ;)

Makefile: Remove -lcrypto.

As libosl.0.1.1 does not depend on libssl any more, linking against libcrypto
is no longer necessary.

Revert "Use ?:= as the assignement operator for PREFIX."

This reverts commit 59af25aafd51b706ed67784aa84332965c31bb26 as
it does not work with all versions of (gnu) make.

Use ?:= as the assignement operator for PREFIX.

This creates a simply expanded variable which is more suitable for
the PREFIX variable.

Set CC to gcc by default.

On systems where cc != gcc, compilation likely fails because we are
using quite some gccisms in adu. So default to gcc but let the user
override the default by setting CC manually.

The straight-forward CC ?= gcc does not work with gnu make as make
assigns CC the default value "cc".

Thanks to Steffen Schmidt for pointing out this shortcoming.

Replace hard-coded gcc by $(CC) in Makefile.deps target.

If the user chooses alternative compiler, she will likely also generate
the dependency file by using that compiler.

Add wrapper for isspace() which is needed for NetBSD.

Transform the database_dir/database_root arg into an absolute path.

adu --create failed badly if called with --database_dir or
--database_root being relative path. This patch fixes the bug.

Unfortunately, it's not completely trivial to obtain the cwd
in a portable and secure manner. The method used in the
new absolute_path() function, while still not bullet-proof, is
the best what one can do if portability is an issue.

INSTALL: Link to the new osl web page rather than to the gitweb page.

Fix typo.

Fix typo in the README file.

Documenatation improvements.

This patch adds an illustration of the user-list mode to README and an
examples section to the man page. Thanks to Klaus Kopec who suggested
this improvement.

Increase width of uid column to 6.

This makes the user summary output look nicer if there are uids > 10000.

Add an example to README.

Suggested by Sebastian Stark.

INSTALL: Add a link to the public git repo.

Merge commit 'athcx/bloom'

Conflicts:

create.c

Fix another typo.

Two typos in the same log message..

Fix typo.

Implement --trailer select option.

It works just like the --header option.

Merge commit 'fml/master'

Conflicts:
adu.ggo
user.c

remove paraslash relict in comment

flush output after each command in interactive mode.

Otherwise "clients" that are not terminals have difficulties
seeing adu's output.

add simple install target for adu and adu.1

new option database-root

If database-root is given, the database dir is computed by
appending the base dir to it. Obviously this works only if
base dir is given, even for select mode.
The needed directory structure is created below database-root
by the new function mkpath(), which recursively creates any
directory needed to copy the structure of base dir. This
has the side-effect that the database dir is always created
for the user (if permissions allow) which wasn't the case
before.

database_dir is now a global variable just like conf and should
be used whenever conf.database_dir_arg was used before.

Merge branch 'master' into bloom

Conflicts:
create.c

Make user.c independent of command line options.

Merge branch 'master' into bloom

Remove unused option --config-file.

bloom.c: Remove some unneccessary includes.

Add source code documentation for bloom_insert().

Rename bloom_test_and_insert() to bloom_insert().

In order to keep statistics up to date, we always need to do the test.

Rename bloom_init() to bloom_new().

bloom: Add some source documentation.

Merge branch 'master' into bloom

Document the stat64 compatibility fix.

Merge branch 'master' into bloom

Make it compile on FreeBSD and NetBSD.

off_t on BSD is 64 bit even on 32bit machines, so there are no
special tricks needed to get large file support. In fact, getconf
has no options for large file support and struct stat64 and lstat64()
do not exist on BSD systems. This caused the compilation to fail on
those systems.

Fix this problem by checking for BSD via uname -s in the Makefile. If
uname indicates we're on BSD, then do not use getconf and #define
stat64 and lstat64() to stat and lstat() respectively.

First draft of hard link detection via bloom filters.

Fix a design bug concerning struct user_info.

It was a bad idea to include the accounting data (#files, #dirs,
information about the user only.

So move the accounting data to user_summary_info. This allows
to get rid of the ugly uid_hash_table_sort_idx construct and of
sort_hash_table(). These were only needed because of the broken
design.  We now never sort the hash table but allocate an array of
user_summary_info structures on each query and sort that array instead.

This patch not only simplifies code but also fixes a real bug noted by
Sebastian Stark: If the user_summary was requested more than once in
interactive mode, the old code computed incorrect values because the
above mentioned accounting fields were only initialized once. The new
code gets this right automatically because a fresh array is created
on each query.

print_complete_help_and_die() does not return.

So change the return type to void.

Merge commit 'fml/master'

Some more source code documentation.

Add documentation of parse_select_options().

Fix a compiler warning.

It's kinda bogus, but anyway. It fixes

select.c:532: warning: 'comp' may be used uninitialized in this function

Add documentation of DEFINE_ERRLIST.

Start writing the mainpage for the API reference.

This should also contain a general overview..

Add yet more source code documentation.

Add some more souce code documentation.

Make close_all_tables() static.

It's only used in adu.c.

Merge commit 'fml/master'

string.c: Replace "paralash" by "adu" in source code documentation.

Add source code documentation for run_select_query() and com_select().

Add missing source code documentation of user.c.

This completes the documentation of user.c.

adu-0.0.5.

Merge commit 'meins/master'

Add documentation of the enhancments to --output.

Code for the new features is still missing.

Add \brief command to file header to make doxygen happy.

Remove sha1.c

This was never needed.

user.c: Add some more source code documentation.

Add copyright header and COPYING file.

Cosmetic cleanups.

Remove duplicate whitespaces in documentation.

man2html does not like that.

Implement pattern matching for select mode.

Minor documentation updates/fixes/improvements.

Make the header line configurable.

This replaces the --no-header flag by --header string option. The
argument is the header of the summary/list.

This was not a straight-forward task because the headers for the
user lists contain the uid and the user name. So introduce another
set of format string directives to still allow this.

format.c: Return an empty buffer for empty format strings rather than NULL.