Writing metadata xattr on directories with sticky bit set, FR#94

Even though we may have write access to a directory, if the sticky bit
is set only the owner is allowed to write xattrs.

Add an option that enables behaviour to set the xattr as root, using
the fact that we can distinguish between a failing xattr set request
due to insufficient permissions (EACCES) or the sticky bit (EPERM):

  force xattr with sticky bit = yes|no (default: no)

Signed-off-by: Ralph Boehme <rb@sernet.de>

config: add utility function vdgoption_bool()

Signed-off-by: Ralph Boehme <rb@sernet.de>

library versioning: update to 16:0:0

Signed-off-by: Ralph Boehme <rb@sernet.de>

Hangs in Netatalk which causes it to stop responding to connections

Hangs in Netatalk which causes it to stop responding to
connections. The master afpd process gets stuck in a poll loop, being
repeatedly notified that there are connections on its socket, but
never actually doesn anything with them.

Analysis with gdb revealed that the dat astructure dealing with the
main AFP socket and the IPC client sockets was smashed. This could
happen because the function fdset_add_fd() doesn't do bound checking
itself but relied on other parts of the code that enforce a connection
limit. Unfortunately, for low-level AFP connections that don't result
in a full AFP login these checks come too late resulting in a buffer
overflow.

Add a bound check. While we're at it, rewrite the fdset code to use a
full blown data structure 'struct asev' encapsultating the
implementation details.

Fixes bug #572.

Signed-off-by: Ralph Boehme <rb@sernet.de>

version: 3.1.4dev

Signed-off-by: Ralph Boehme <rb@sernet.de>

Release: version 3.1.3

Signed-off-by: Ralph Boehme <rb@sernet.de>

abigen: fix for gdb syms error

Signed-off-by: Ralph Boehme <rb@sernet.de>

gitignore: test-driver

Signed-off-by: Ralph Boehme <rb@sernet.de>

manpages: generate from XML

Signed-off-by: Ralph Boehme <rb@sernet.de>

Add advanced option "chmod request" controlling ACLs

chmod request = ignore | preserve (default) | simple

* ignore = UNIX chmod() requests are completely ignored
* preserve = preserve ZFS ACEs for named users and groups or POSIX ACL
  group mask
* simple = just to a chmod() as requested without any extra steps

Signed-off-by: Ralph Boehme <rb@sernet.de>

Use realpath_safe() for getting the absolute path

Authored-by: gaaronk
Reviewed-by: Ralph Boehme <rb@sernet.de>

ad: path detection in getvolbypath(), bug #570

The fix for bug #563 broke ad file utilities, bug #570.

Authored-by: gaaronk
Reviewed-by: Ralph Boehme <rb@sernet.de>

Resource fork conversion from AppleDouble v2 broken, bug #568

Converting from AppleDouble v2 to ea may corrupt the resource fork. In
some circumstances an offset calculation is wrong resulting in corrupt
resource forks after the conversion.

Ensure the resource fork AppleDouble entry offset is always set up and
we don't rely on the manual offset calculation.

Skip ADEID_RFORK in ad_copy_header() and allow copying of AppleDouble
entry that have an initial zero size in the destination adouble.

Signed-off-by: Ralph Boehme <rb@sernet.de>

Merge branch 'branch-netatalk-3-1' of netafp.com:git/netatalk into branch-netatalk-3-1

Conflicts:
NEWS

Fix a crash when accessing ._ files, bug #564

At least FreeBSD is picky in that it requires PROT_READ if we're going
to read a mmap()ed memory region mapped, only specifying PROT_WRITE
doesn't imply PROT_READ. Other OSen seem to handle this differently.

Also add a missing pointer reinitialisation to the buf pointer.

Reported-by: Sean Eric Fagan
Signed-off-by: Ralph Boehme <rb@sernet.de>

NEWS: Fix FR number

Signed-off-by: Ralph Boehme <rb@sernet.de>

Delete 0 byte size resource fork ._ files

Signed-off-by: Ralph Boehme <rb@sernet.de>
Change-Id: ef5f88f5f2cefa75cf20d0ee447b4e20579a2f12

Refactoring: move forkpath computation to function scope

Preperation for new feature to delete zero byte sized resource forks.

Signed-off-by: Ralph Boehme <rb@sernet.de>
Change-Id: ef5f88f5f2cefa75cf20d0ee447b4e20579a2f12

Refactoring: move struct dir and dirlookup() to function scope

Preperation for new feature to delete zero byte sized resource forks.

Signed-off-by: Ralph Boehme <rb@sernet.de>
Change-Id: ef5f88f5f2cefa75cf20d0ee447b4e20579a2f12

Memory overflow caused by 'basedir regex', bug #567

Authored-by: Takao Shimayoshi
Reviewed-by: Ralph Boehme <rb@sernet.de>

Update Unicode support to version 7.0.0

Authored-by: HAT <hat@fa2.so-net.ne.jp>
Reviewed-by: Ralph Boehme <rb@sernet.de>

Unarchiving certain ZIP archives fails, bug #569

The current code that checks whether the client is allowed to use
pathnames starting with ._ checks whether the file is an AppleDouble
file, and if it is and the filler does not contain the magic string
"Mac OS X", access to the file is denied. This results in the
following sequence of steps:

* client created empty file "._NAME"
* client wants to stat the newly created file "._NAME"
* server checks whether access to the file is allowed
* check fails

Modify check in ad_valid_header_osx() to allow access to all .-
AppleDouble files where the filler string is not "Netatalk", which is
the magic string Netatalk puts there.

Signed-off-by: Ralph Boehme <rb@sernet.de>

fix typo

Spotlight: "spotlight expr = false" broke the default Spotlight query

The default Spotlight query a Mac uses is

  *=="test*"cwd || kMDItemTextContent=="test*"cwd

which is of course using a logic expression. The parser has an extra
rule for catching this (MATCH or MATCH) for the purpose of munging
both expression into a single fts:match SPARQL query. Which means
we shouldn't check whether logic expressions are enabled or not.

Signed-off-by: Ralph Boehme <rb@sernet.de>

Spotlight: new options for controlling query behaviour

Add three options that allow fine grainted control over the resulting
SPARQL queries:

"sparql results limit = NUMBER", default is unlimited
Useful for limiting the result set for very large systems

"spotlight attributes = STRING", useful for limiting queries to
attributes that are indexed, many are not.

"spotlight expr = BOOLEAN", default: yes, useful for disabling the use
of complex queries that will take too long to complete and hang Tracker.

Signed-off-by: Ralph Boehme <rb@sernet.de>

Spotlight: SPARQL query optimisations

Use tracker:uri-is-descendant FILTER instead of regex for filtering
results on the base path of the volume.

Signed-off-by: Ralph Boehme <rb@sernet.de>

Bump version to 3.1.3dev

Remove uniconv

Prepare 3.1.2 releas

Add generated files from yacc and flex

Spotlight: SPARQL query optimisation

Avoid DISTINCT in SPARQL queries and add a LIMIT which is arbitrarily
set to 100 for now.

Signed-off-by: Ralph Boehme <rb@sernet.de>

New options that control whether dbus and Tracker are started

'start dbus' and 'start tracker', both default to yes, FR#91

Signed-off-by: Ralph Boehme <sloowfranklin@gmail.com>

fd leak when using appledouble = v2, bug #554

With appledouble = v2 in afp.conf there is an fd leak in certain
circumstances while using the Finder, from Daniel Hazelbaker.

Signed-off-by: Ralph Boehme <rb@sernet.de>

getvolbypath returns incorrect volume, bug #563

getvolbypath() would match a given path "/foo/barbaz" with an existing
volume path of "/foo/bar" because the strings where compared with
strnlen() where n is the length of "/foo/bar", ie we did a substring
match.

I faintly remember there was a reason for this, but I couldn't come up
with any sensible configuration of usage of dbd that would not also
work by simply using strcmp() instead of strmcmp().

Of course, in order for dbd to work with paths that included a
trailing slash, there's a need to adding some slash stripping code,
because our code (somehow) ensures (hopefully!) that in struct
vol.v_path is always stored with trailinsh slashes stripped.

Signed-off-by: Ralph Boehme <rb@sernet.de>

dbd: two fixes for new user option

* initialize user variable with NULL
* add -u to option summary line

Signed-off-by: Ralph Boehme <rb@sernet.de>

Support for using $u username variable in AFP volume definitions

* Prepare CNID backend dbd for user variable expansion
* Store a struct vol pointer in the CNID db handles.
* Pass three arguments to the CNID daemon:
  - the afp.conf volume section name
  - the volume path
  - an optional username

* Rename several struct elements by prefixing them with a struct tag,
  helps catching bad or missing assignments.

* Disable uniconv, doesn't use laod_volumes() and thus wont work with
  the new CNID dbd backend.

From FR#90.

Signed-off-by: Ralph Boehme <rb@sernet.de>

xattrs on BSD, bug #562

Signed-off-by: Ralph Boehme <rb@sernet.de>

quotas: improve Linux quota behaviour

Signed-off-by: Ralph Boehme <rb@sernet.de>

Merge branch 'sharemodes-bug560' into branch-netatalk-3-1

locking: put the Solaris share reservation after our locking stuff

Fix for Bug 560: put the Solaris share reservation after our locking stuff.
Note that this still leaves room for a race condition where between placing our own
locks above and putting the Solaris share move below another client puts a lock.
We then end up with set locks from above and return with an error code, the proper
fix requires a sane cleanup function for the error path in this function.

Signed-off-by: Ralph Boehme <rb@sernet.de>

Spotlight: add SPAQRL mapping for kMDItemLastUsedDate

Signed-off-by: Ralph Boehme <rb@sernet.de>

fix typo

createvol -> creatvol

debian initscript: add 0 and 6 to Default-Stop. debian-bug#745520

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=745520

Only run dbus-daemon if Spotlight is enabled, bug #556

Remove tracker miner code

tracker_miner_manager_new_full() was reported to be potentially
locking up for an unknown reason.

As we diddn't use the miner anyway for other reasons (see comment in
the code), simply #ifdef 0 the whole code.

add pwent->pw_dir to logmessage.

inspired by Bug #557

Remove misleading logmessage added in previous commit e4e8f3a8

Improve reliability of afpd child handler

Spotlight: file modification date, bug #545

The Spotlight module misleadingly updated a files modification date
when the client requested an "kMDItemLastUsedDate" attribute update.

It's questionable whether we're expected at all to map and modify
attributes to filesystem metadata, but my guess is we are.

As a fix, use "kMDItemFSContentChangeDate" attribute instead of
"kMDItemLastUsedDate".

Fixes bug #545.

Add missing parenthesis

Fix typo and remove unused variables

Option "vol dbpath" was broken in 3.1.1

Signed-off-by: Ralph Boehme <rb@sernet.de>

Bump version

Update release notes

Update libatalk so version

Bump version to 3.1.1

Don't flood log with failed sys_set_ea() messages

Run dbpath through variable substitution

Add a flag to load_volumes() that controls load behaviour

Don't print message "Ignoring ._file" for every ._ file, bug #552

Fix a crash in of_closefork(), bug #551

Apparently dirlookup() might return NULL when trying to query the did
of an open fork:

    dir = dirlookup(ofork->of_vol, ofork->of_did);

Add a NULL check before using the result.

Generate manpage from XML

conversion for OS X ._ AppleDouble files, bug #550

Add automatic on-access conversion for ._ AppleDouble files created by
OS X to Netatalk's AppleDouble format.

OS X .AppleDouble files most of the times contain a FinderInfo
AppleDouble entry with a size larger then 32 bytes, containing packed
xattrs. Netatalk fails to cleanly deal with these and logs them as
"bogus entry id 9".

dbd: remove orphaned ._ AppleDouble files. Bug #549.

veto files: syntax clarification

Add file pathname to logmessage parse_entries: bogus eid. FR#87

update manpage of afp.conf

add dbus daemon.
sort.

macusers showed root user. Bug #495.

Removing ressource fork AppleDouble fails, bug #542

When the AFP client uses FPSetForkParms to set the ressource fork
length to 0, Netatalk will call unlink() in ad_rtruncate() in
libatalk/adouble/ad_write.c in order to to unlink the files
AppleDouble ressource fork.

The function ad_rtruncate() gets just the filename as path from the
caller, so obviously at this point the caller should have ensured that
it had called chdir() to the enclosing directory of the file.

Fixes bug #542.

afpd -V show Spotlight support

Registering with mDNS crashed, bug #540

Fix for incomplete AFP 3.4 commit 70e3fcd8744752d6de0b93d089f41c28dca30bde

Return "no such item" error when non-existing EAs are requested.

Merge pull request #12 from hat001/avahi-workaround-3-1

Workaround for a problem which cannot be advertized by Avahi. Bug #541.

Workaround for a problem which cannot be advertized by Avahi. Bug #541.

Add support for AFP 3.4

Update NEWS

Fix a bug where only the first configured UAM was loaded

Caused by strtok() not being async signal safe, use strtok_r() instead.
Fixes bug #537.

Update NEWS

Fix ressource fork name conversion

Function was using ad_path() which returns a pointer to a static
string twice in the rename() functions.

Fixes bug #534.

Add asprint() compatibility function for systems lacking it

Update version for 3.1.1 development

Add libatalk ABI for 3.1.0 and bump library version leaving room for 3.0 series

Update version and release notes for 3.1.0 release

Add exit code

Use dedicated exit code for AFP connections that were dropped
by the client right after the TCP handshake.

Merge remote-tracking branch 'origin/branch-netatalk-3-0' into develop

Conflicts:
NEWS
VERSION
doc/www/ReleaseNotes

Build fixes for the Kerberos UAM

o missing Kerberos includes and libraries
o conditional code that handled the case when Kerberos is not available
  was c/p from etc/afp/status.c and wasn't updated for the changes introduced
  in the bugfix for #531

Update version and news for 3.0.7

Merge branch 'branch-netatalk-3-0' into release-3.0.6

Update release notes for 3.0.6

Bump ABI version and include ABI file for 3.0.5 and 3.0.6

Set version to 3.0.6

Merge remote-tracking branch 'origin/branch-netatalk-3-0' into develop

Remove obsolete compatibility function declarations for snprintf() and vsnprintf()

We were providing function declarations without ever providing function implementations.
Also, for some reason, this caues compilation issues on older Linux versions.

Add MySQL CNID backend to the manual

Add MySQL CNID backend options to the manpage

New MySQL CNID backend

o add configure check --with-mysql-config=PATH that searches for
  the mysql-config binary
o add new MySQL CNID subdirectory and compilation infrastructure
o modify CNID header files for new backend
o add private CNID MySQL header file for private data
o add mysql config options
o add MySQL args to cnid_open()
o add support for MySQL CNID backend to dbd

Add MySQL libraries as deps to executables:
workaround libtool bug where the rpath of a dependent library (ie mysql) of a libtool
library (ie libatalk) gets stripped from the link flags when linking the final
executable, even though the library (mysql) is a non default OS dir.

Add vasprintf compatibility function

Add recvfile support with splice() on Linux

o Global option "recvfile" controls whether splice() is used.
  Ddefaults to false.
o Global option "splice size" limits the amount of data spliced.
  Defaults to 64k.

Merge remote-tracking branch 'origin/branch-netatalk-3-0' into develop

Conflicts:
configure.ac
libatalk/Makefile.am

Merge remote-tracking branch 'origin/branch-netatalk-3-0' into develop