]> arthur.barton.de Git - netatalk.git/log
netatalk.git
11 years agoMerge remote-tracking branch 'origin/develop' into spotlight
Ralph Boehme [Tue, 2 Apr 2013 16:13:43 +0000 (18:13 +0200)]
Merge remote-tracking branch 'origin/develop' into spotlight

Conflicts:
VERSION

11 years agoConvert charset of query
Ralph Boehme [Tue, 2 Apr 2013 15:44:16 +0000 (17:44 +0200)]
Convert charset of query

11 years agoOpening files without metadata EA may result in broken metadata EA
Ralph Boehme [Wed, 27 Mar 2013 16:18:22 +0000 (17:18 +0100)]
Opening files without metadata EA may result in broken metadata EA

ad_open_hf_ea() calls ad_flush() after initializing the new metadata,
but in ad_flush() ad_flush_ea() wasn't called because we didn't
yet increment the refcount.
Fix this by increasing the refcount in ad_open_hf() before calling
into ad_open_hf_ea(), decrementing it again in case of an error.

This ensure a valid metadata EA is created, but it still doesn't
set the CNID field values. Therefor in afp_openfork() we must check
whether the ad_open() created a metadata EA and then set the CNID
info.

Finally, check for malformed metadata EAs and delete them.

Fixes bug #510.

11 years agoChange log facility
Ralph Boehme [Wed, 27 Mar 2013 14:38:19 +0000 (15:38 +0100)]
Change log facility

11 years agoAdd 3.0.4 news
Ralph Boehme [Tue, 26 Mar 2013 17:46:24 +0000 (18:46 +0100)]
Add 3.0.4 news

11 years ago3.0.4dev
Frank Lahm [Tue, 26 Mar 2013 09:43:00 +0000 (10:43 +0100)]
3.0.4dev

11 years agoPrepare 3.0.3 release
Frank Lahm [Tue, 26 Mar 2013 09:17:03 +0000 (10:17 +0100)]
Prepare 3.0.3 release

11 years agoFix "ad ls" segfault if requested object is not in an AFP volume
Ralph Boehme [Fri, 22 Mar 2013 12:53:10 +0000 (13:53 +0100)]
Fix "ad ls" segfault if requested object is not in an AFP volume

Fixes bug #496.

11 years agoAdd afpstats manpage and update afp.conf
Ralph Boehme [Sun, 24 Mar 2013 08:35:19 +0000 (09:35 +0100)]
Add afpstats manpage and update afp.conf

11 years agonew configure option --with-init-dir=PATH
HAT [Fri, 22 Mar 2013 15:12:15 +0000 (00:12 +0900)]
new configure option --with-init-dir=PATH

11 years agoOnly configure libevent when used
Ralph Boehme [Fri, 22 Mar 2013 13:37:06 +0000 (14:37 +0100)]
Only configure libevent when used

11 years agoSetting POSIX ACLs on Linux
Frank Lahm [Thu, 14 Mar 2013 18:54:52 +0000 (19:54 +0100)]
Setting POSIX ACLs on Linux

The VFS module for setting ACLs on Linux returned an error when
try to set an ACL for a file that doesn't have a ressource fork.

Fix this by checking for ENOENT errno and then returning with 0
from the VFS stack.

11 years agoFix a debug message
Ralph Boehme [Fri, 22 Mar 2013 12:11:48 +0000 (13:11 +0100)]
Fix a debug message

11 years agoCan't set Finder label on symlinked folder with "follow symlinks = yes"
Ralph Boehme [Thu, 21 Mar 2013 16:21:59 +0000 (17:21 +0100)]
Can't set Finder label on symlinked folder with "follow symlinks = yes"

Fixes bug #508.

11 years agoIncrease IO size when sendfile() is not used
Ralph Boehme [Tue, 19 Mar 2013 14:26:03 +0000 (15:26 +0100)]
Increase IO size when sendfile() is not used

If sendfile() support is disabled, the current code results in an
IO size of 8k for transferring requested data to the client. Eg:
- client request to read DSI quantum size bytes (~250k) from a file
- afpd process loops read(8k, file) -> send(8k, client) until all
  250k have been transferred
This should be modified to read the whole 250k in one swoop from the
file to a buffer and then send the whole buffer in one swoop to the
client.

From FR #76.

11 years agoFix ressource fork refcounting
Ralph Boehme [Tue, 19 Mar 2013 12:15:10 +0000 (13:15 +0100)]
Fix ressource fork refcounting

11 years agoread_file() clobbered the AFP error code
Ralph Boehme [Tue, 19 Mar 2013 11:18:34 +0000 (12:18 +0100)]
read_file() clobbered the AFP error code

11 years agoUse FreeBSD sendfile() capability to send protocol header
Frank Lahm [Fri, 15 Mar 2013 08:19:40 +0000 (09:19 +0100)]
Use FreeBSD sendfile() capability to send protocol header

When transmitting file contents to a client with sendfile(), on
every platform other then Solaris (where we use sendfilev()), we
send the AFP the protocol header with an additional send with
MSG_MORE before calling sendfile().
FreeBSD sendfile() supports sending protocol header (and trailer)
data similarly to Solaris sendfilev().

11 years agoMerge remote branch 'github/develop' into develop
Frank Lahm [Fri, 15 Mar 2013 12:41:39 +0000 (13:41 +0100)]
Merge remote branch 'github/develop' into develop

11 years agoMerge pull request #25 from hat001/of_findname
franklahm [Fri, 15 Mar 2013 12:40:45 +0000 (05:40 -0700)]
Merge pull request #25 from hat001/of_findname

of_findname(): missing argument

11 years agoof_findname(): missing argument
HAT [Fri, 15 Mar 2013 11:32:21 +0000 (20:32 +0900)]
of_findname(): missing argument

11 years agoMerge remote-tracking branch 'origin/develop' into spotlight
Ralph Boehme [Tue, 12 Mar 2013 23:12:58 +0000 (00:12 +0100)]
Merge remote-tracking branch 'origin/develop' into spotlight

11 years agoafpstats-service.xml afpstats_service_glue.h were missing in distribution
Frank Lahm [Mon, 11 Mar 2013 17:42:23 +0000 (18:42 +0100)]
afpstats-service.xml afpstats_service_glue.h were missing in distribution

11 years agoPermissions of ._ AppleDouble rfork after conversion from v2 to ea.
Frank Lahm [Mon, 11 Mar 2013 16:19:28 +0000 (17:19 +0100)]
Permissions of ._ AppleDouble rfork after conversion from v2 to ea.

Fixes bug #505.

11 years agosys_lsetxattr error messages when mounting read only filesystem
Ralph Boehme [Tue, 5 Mar 2013 16:40:22 +0000 (17:40 +0100)]
sys_lsetxattr error messages when mounting read only filesystem

Fixes bug #504.

11 years agoafpd -V show dbus and dtrace
HAT [Sun, 10 Mar 2013 15:23:30 +0000 (00:23 +0900)]
afpd -V show dbus and dtrace

11 years agoConversion from adouble v2 to ea for directories
Frank Lahm [Sat, 9 Mar 2013 13:23:33 +0000 (14:23 +0100)]
Conversion from adouble v2 to ea for directories

Fixes bug #500.

11 years agoSolaris fixes for bugs #501 and #502
Ralph Boehme [Sat, 9 Mar 2013 12:50:37 +0000 (13:50 +0100)]
Solaris fixes for bugs #501 and #502

11 years agoFile's ressource fork can't be read if metadata EA is missing
Frank Lahm [Fri, 8 Mar 2013 17:29:13 +0000 (18:29 +0100)]
File's ressource fork can't be read if metadata EA is missing

Fixes bug #501.

11 years agoFix an issue caused by read-only ._ ressource forks
Frank Lahm [Thu, 7 Mar 2013 18:03:30 +0000 (19:03 +0100)]
Fix an issue caused by read-only ._ ressource forks

Fixes bug #502.

11 years agoAdd IPC commands to string array
Frank Lahm [Wed, 6 Mar 2013 18:10:06 +0000 (19:10 +0100)]
Add IPC commands to string array

11 years agoCheck for dbus_g_bus_get_private(), older dbus-glib versions seem to miss it
Frank Lahm [Wed, 6 Mar 2013 17:35:15 +0000 (18:35 +0100)]
Check for dbus_g_bus_get_private(), older dbus-glib versions seem to miss it

11 years agoSet index-single-directories to empty list
Ralph Boehme [Tue, 5 Mar 2013 11:30:11 +0000 (12:30 +0100)]
Set index-single-directories to empty list

11 years agoReenable tracker-miner interaction
Ralph Boehme [Tue, 5 Mar 2013 11:26:18 +0000 (12:26 +0100)]
Reenable tracker-miner interaction

11 years agoMerge remote-tracking branch 'origin/develop' into spotlight
Ralph Boehme [Tue, 5 Mar 2013 10:52:11 +0000 (11:52 +0100)]
Merge remote-tracking branch 'origin/develop' into spotlight

11 years agoPossible fixes to afpstats dbus issue
Ralph Boehme [Tue, 5 Mar 2013 08:32:28 +0000 (09:32 +0100)]
Possible fixes to afpstats dbus issue

Initialize afpstats before calling configinit in main() thus ensuring
dbus glib thread initilisation is performed as we need it.

Use private dbus connection from dbus_g_bus_get_private() which should
help with issues seen where our avahi dbus thread crashed.

11 years agoMerge remote-tracking branch 'origin/develop' into spotlight
Ralph Boehme [Mon, 4 Mar 2013 20:50:58 +0000 (21:50 +0100)]
Merge remote-tracking branch 'origin/develop' into spotlight

Conflicts:
include/atalk/globals.h

11 years agoFix check for unsupported attributes
Ralph Boehme [Mon, 4 Mar 2013 20:40:17 +0000 (21:40 +0100)]
Fix check for unsupported attributes

11 years agoAlways install dbus-session.conf
Ralph Boehme [Mon, 4 Mar 2013 20:10:13 +0000 (21:10 +0100)]
Always install dbus-session.conf

11 years agoNew option 'afpstats'
Ralph Boehme [Mon, 4 Mar 2013 16:45:19 +0000 (17:45 +0100)]
New option 'afpstats'

11 years agoMostly revert c86c729e2190f1f11a7ed7633b2e5d1fdb9f6ff6 and call Tracker stuff as...
Ralph Boehme [Thu, 28 Feb 2013 04:08:30 +0000 (05:08 +0100)]
Mostly revert c86c729e2190f1f11a7ed7633b2e5d1fdb9f6ff6 and call Tracker stuff as root

11 years agoTry to remove non existent files from the Tracker index
Ralph Boehme [Wed, 27 Feb 2013 15:59:14 +0000 (16:59 +0100)]
Try to remove non existent files from the Tracker index

11 years agoFix dbus session config to allow any user to connnect
Ralph Boehme [Wed, 27 Feb 2013 09:50:48 +0000 (10:50 +0100)]
Fix dbus session config to allow any user to connnect

11 years agoChange pkg-config check for dbus-glib
Ralph Boehme [Wed, 27 Feb 2013 07:57:55 +0000 (08:57 +0100)]
Change pkg-config check for dbus-glib

11 years agostp4
Ralph Boehme [Mon, 25 Feb 2013 17:31:36 +0000 (18:31 +0100)]
stp4

11 years agoMerge remote-tracking branch 'origin/develop' into spotlight
Ralph Boehme [Mon, 25 Feb 2013 17:07:18 +0000 (18:07 +0100)]
Merge remote-tracking branch 'origin/develop' into spotlight

11 years agoFix distcheck error caused by pkgconfig stuff
Ralph Boehme [Mon, 25 Feb 2013 17:07:01 +0000 (18:07 +0100)]
Fix distcheck error caused by pkgconfig stuff

11 years agoMerge remote-tracking branch 'origin/develop' into spotlight
Ralph Boehme [Mon, 25 Feb 2013 11:41:29 +0000 (12:41 +0100)]
Merge remote-tracking branch 'origin/develop' into spotlight

11 years agoDisable Tracker miner stuff, hang since adding dbus afpstats
Ralph Boehme [Mon, 25 Feb 2013 11:41:09 +0000 (12:41 +0100)]
Disable Tracker miner stuff, hang since adding dbus afpstats

11 years agoAdd PATH to helpstring
Ralph Boehme [Mon, 25 Feb 2013 11:12:48 +0000 (12:12 +0100)]
Add PATH to helpstring

11 years agoAdd tracker.conf to EXTRA_DIST and fix source install path
Ralph Boehme [Mon, 25 Feb 2013 10:47:11 +0000 (11:47 +0100)]
Add tracker.conf to EXTRA_DIST and fix source install path

11 years agoRemove unneeded include
Ralph Boehme [Sun, 24 Feb 2013 09:15:05 +0000 (10:15 +0100)]
Remove unneeded include

11 years agoconfigure summary show paths
HAT [Fri, 22 Feb 2013 14:30:10 +0000 (23:30 +0900)]
configure summary show paths

11 years agoRemove duplicated test
Ralph Boehme [Fri, 22 Feb 2013 11:56:43 +0000 (12:56 +0100)]
Remove duplicated test

11 years agoAdd a distinct variable for storing result of checks
Frank Lahm [Fri, 22 Feb 2013 09:53:55 +0000 (10:53 +0100)]
Add a distinct variable for storing result of checks

11 years agoReload groups when reloading volumes
Ralph Boehme [Thu, 21 Feb 2013 14:19:02 +0000 (15:19 +0100)]
Reload groups when reloading volumes

Reload the user groups every time the volume configuration changes
and also to call initgroups() again with the possibly changed groups.

FR #71

11 years agoAdd checking for and linking with libgthread
Frank Lahm [Thu, 21 Feb 2013 13:48:09 +0000 (14:48 +0100)]
Add checking for and linking with libgthread

11 years agoAdd dtrace provider 'afp' and dtrace probes
Ralph Boehme [Mon, 11 Feb 2013 13:42:12 +0000 (14:42 +0100)]
Add dtrace provider 'afp' and dtrace probes

There are three dtrace probes:
- all AFP functions, passing the function name as argument
- AFP read and write, passing the IO size as argument
- all CNID functions

In order to be able to simply differintiate between disk IO and
network IO, replace all occurences of read and write on sockets
with send/recv, and replace the use of writev in the logging code
with two calls to write.
As a result, using dtrace probes for read() and write() syscalls
can be used to gather metrics for disk IO, while probing send(),
recv() and writev() probe network IO.

11 years agoAFP statistics via dbus IPC
Ralph Boehme [Tue, 5 Feb 2013 16:25:27 +0000 (17:25 +0100)]
AFP statistics via dbus IPC

Fix PAM config installation. make distcheck was failing on Solaris
in uninstall, because the uninstall target ran some `rm -f PAMFILE`
where PAMFILE is hardcoded to /etc/pam.d. The resulting EPERM
causes the make target to fail.

11 years agoSeveral fixes in ad_conv_dehex()
Ralph Boehme [Tue, 19 Feb 2013 16:35:04 +0000 (17:35 +0100)]
Several fixes in ad_conv_dehex()

Testing whether we need to perform dehexing was done by a simple
strchr(':'), thus names only containing a single ':' would trigger
dehexing even though they might not contain the relevant strings
":2e" or :2f".

Also fix a ressource leak in bfindreplace() and simplify the
check whether a rename took place in enumerate.c.

11 years agotypo in netatalk.c
HAT [Fri, 15 Feb 2013 12:13:35 +0000 (21:13 +0900)]
typo in netatalk.c

11 years agoAdd checks for required packages
Ralph Boehme [Fri, 15 Feb 2013 10:32:15 +0000 (11:32 +0100)]
Add checks for required packages

11 years agoUser homes directory names
Ralph Boehme [Fri, 15 Feb 2013 10:19:50 +0000 (11:19 +0100)]
User homes directory names

Fix an issue with user homes when user home directory has not the
same name as the username.

Fixes bug #497.

11 years agoMerge pull request #20 from hat001/redhatinit
franklahm [Thu, 14 Feb 2013 14:47:17 +0000 (06:47 -0800)]
Merge pull request #20 from hat001/redhatinit

update RedHat initscript

11 years agoupdate RedHat initscript
HAT [Thu, 14 Feb 2013 14:27:29 +0000 (23:27 +0900)]
update RedHat initscript

"graceful" is removed because SIGQUIT is gone.
"reload" use SIGHUP.
"condrestart" is added because spec file requires it.

11 years agoafpd/file: fix check in deletefile
Riccardo Magliocchetti [Wed, 13 Feb 2013 21:18:41 +0000 (22:18 +0100)]
afpd/file: fix check in deletefile

AD_RSRC_OPEN(ad) is defined as:

(((ad)->ad_reso_refcount) && (ad_reso_fileno(ad) >= 0))

So there's no way it can returns -1

Reported by coverity as CID #762087.

11 years agolibatalk: avoid double close in ad_openat
Riccardo Magliocchetti [Wed, 13 Feb 2013 20:58:00 +0000 (21:58 +0100)]
libatalk: avoid double close in ad_openat

Partly revert 8852315e574879a9548c31f84c6eaea59f1db90f

Reported by Coverity as CID #980995.

11 years agoafpd/volume: avoid read of unitialized pointer
Riccardo Magliocchetti [Wed, 13 Feb 2013 20:54:01 +0000 (21:54 +0100)]
afpd/volume: avoid read of unitialized pointer

Reported by Coverity as CID #980994.

11 years agoCouldn't create folders on FreeBSD 9.1 ZFS fileystems
Frank Lahm [Wed, 13 Feb 2013 16:27:52 +0000 (17:27 +0100)]
Couldn't create folders on FreeBSD 9.1 ZFS fileystems

Fixes bug #491.

11 years agoDeclare as static
Ralph Boehme [Wed, 13 Feb 2013 15:00:23 +0000 (16:00 +0100)]
Declare as static

11 years agoMerge remote-tracking branch 'origin/develop' into spotlight
Ralph Boehme [Sun, 24 Feb 2013 17:37:30 +0000 (18:37 +0100)]
Merge remote-tracking branch 'origin/develop' into spotlight

11 years agoAdd support for searches for folders
Ralph Boehme [Fri, 22 Feb 2013 11:53:54 +0000 (12:53 +0100)]
Add support for searches for folders

11 years agoMerge remote-tracking branch 'origin/develop' into spotlight
Ralph Boehme [Thu, 21 Feb 2013 17:43:15 +0000 (18:43 +0100)]
Merge remote-tracking branch 'origin/develop' into spotlight

Conflicts:
config/Makefile.am
etc/afpd/afp_dsi.c
etc/afpd/auth.c
include/atalk/Makefile.am

11 years agoFixes
Ralph Boehme [Wed, 20 Feb 2013 13:53:36 +0000 (14:53 +0100)]
Fixes

11 years agoAdd tracker loglevel config option
Ralph Boehme [Tue, 19 Feb 2013 13:19:42 +0000 (14:19 +0100)]
Add tracker loglevel config option

11 years agoAdd a tracker.cfg file
Ralph Boehme [Tue, 19 Feb 2013 10:19:21 +0000 (11:19 +0100)]
Add a tracker.cfg file

11 years agoAdd filetype searching and searching by dates
Ralph Boehme [Tue, 19 Feb 2013 09:56:47 +0000 (10:56 +0100)]
Add filetype searching and searching by dates

11 years agoBasic Tracker RDF queries working
Ralph Boehme [Mon, 18 Feb 2013 16:43:53 +0000 (17:43 +0100)]
Basic Tracker RDF queries working

11 years agoRename Tracker 0.6 module to RDF
Ralph Boehme [Mon, 18 Feb 2013 07:15:20 +0000 (08:15 +0100)]
Rename Tracker 0.6 module to RDF

11 years agoMerge branch 'develop' into spotlight
Ralph Boehme [Thu, 14 Feb 2013 15:24:11 +0000 (16:24 +0100)]
Merge branch 'develop' into spotlight

Conflicts:
VERSION
configure.ac

11 years agoMerge branch 'spotlight' of netafp.com:git/netatalk into spotlight
Ralph Boehme [Thu, 14 Feb 2013 15:17:03 +0000 (16:17 +0100)]
Merge branch 'spotlight' of netafp.com:git/netatalk into spotlight

11 years agoMerge pull request #18 from xrmx/coverity3
franklahm [Tue, 12 Feb 2013 06:01:29 +0000 (22:01 -0800)]
Merge pull request #18 from xrmx/coverity3

More issues reported by Coverity

11 years agolibatal/vfs: stats a path before checking if it is a directory
Riccardo Magliocchetti [Mon, 11 Feb 2013 21:43:53 +0000 (22:43 +0100)]
libatal/vfs: stats a path before checking if it is a directory

Reported by Coverity as CID #201301.

11 years agoafpd/directory: avoid unitialized pointer read
Riccardo Magliocchetti [Mon, 11 Feb 2013 21:35:21 +0000 (22:35 +0100)]
afpd/directory: avoid unitialized pointer read

Initialize fullpath to null so that in case of error before its
allocation we don't call bdestroy on random memory.

Reported by Coverity as CID #201284.

11 years agocnid_dbd: fix resource leak
Riccardo Magliocchetti [Mon, 11 Feb 2013 21:31:24 +0000 (22:31 +0100)]
cnid_dbd: fix resource leak

Reported by Coverity as CID #968567.

11 years agoafpd/colume: fix a couple of memory leaks
Riccardo Magliocchetti [Mon, 11 Feb 2013 21:18:01 +0000 (22:18 +0100)]
afpd/colume: fix a couple of memory leaks

Reported by Coverity as CID #201199, #201200.

11 years agoafpd/filedir: fix memory leak
Riccardo Magliocchetti [Mon, 11 Feb 2013 21:11:46 +0000 (22:11 +0100)]
afpd/filedir: fix memory leak

Reported by Coverity as CID #201196.

11 years agoafpd/file: fix memory leak
Riccardo Magliocchetti [Mon, 11 Feb 2013 21:03:26 +0000 (22:03 +0100)]
afpd/file: fix memory leak

Reported by Coverity as CID #201194.

11 years agosystemd service file use PIDFile and ExecReload
Ralph Boehme [Thu, 7 Feb 2013 11:25:13 +0000 (12:25 +0100)]
systemd service file use PIDFile and ExecReload

Add --with-lockfile=PATH configure option for specifying an
alternative path for the netatalk lockfile.

Implements FR #70.

11 years agoMerge pull request #17 from hat001/staticlibevent
franklahm [Sun, 10 Feb 2013 07:30:24 +0000 (23:30 -0800)]
Merge pull request #17 from hat001/staticlibevent

bundled libevent2 is static

11 years agobundled libevent2 is static
HAT [Sun, 10 Feb 2013 03:27:35 +0000 (12:27 +0900)]
bundled libevent2 is static

11 years agoMerge pull request #16 from xrmx/coverity2
franklahm [Sat, 9 Feb 2013 21:09:50 +0000 (13:09 -0800)]
Merge pull request #16 from xrmx/coverity2

More issues reported by Coverity

11 years agoMerge pull request #15 from xrmx/coverity1
franklahm [Sat, 9 Feb 2013 21:09:02 +0000 (13:09 -0800)]
Merge pull request #15 from xrmx/coverity1

Fix some issues reported by Coverity scan

11 years agoafpd/hash: return error if we can't allocate the hash
Riccardo Magliocchetti [Sat, 9 Feb 2013 11:11:19 +0000 (12:11 +0100)]
afpd/hash: return error if we can't allocate the hash

Instead of possible dereference of null.

Reported by Coverity as CID #968565.

11 years agoafpd/acls: fix dereference before null check error
Riccardo Magliocchetti [Sat, 9 Feb 2013 11:07:28 +0000 (12:07 +0100)]
afpd/acls: fix dereference before null check error

Reported by Coverity as CID #968569.

11 years agoafpd: fix out of bounds write
Riccardo Magliocchetti [Sat, 9 Feb 2013 10:57:46 +0000 (11:57 +0100)]
afpd: fix out of bounds write

Reported by Coverity as CID #762111.

11 years agocnid_metad: avoid out of bounds read
Riccardo Magliocchetti [Sat, 9 Feb 2013 10:49:42 +0000 (11:49 +0100)]
cnid_metad: avoid out of bounds read

The check of i < MAXVOLS should be moved before referencing srv[i].

Reported by Coverity as CID #762114.

11 years agolibatalk: Fix a couple of issues in ad_openat()
Riccardo Magliocchetti [Sat, 9 Feb 2013 10:36:29 +0000 (11:36 +0100)]
libatalk: Fix a couple of issues in ad_openat()

Reported by coverity as CID #762151, #762127.

11 years agoafpd/fce_api: avoid null pointer dereference
Riccardo Magliocchetti [Fri, 1 Feb 2013 18:33:59 +0000 (19:33 +0100)]
afpd/fce_api: avoid null pointer dereference

Move path NULL check before it is actually referenced.

Reported by Coverity as CID #968570.

11 years agoafpd/fork: avoid use after free on of_closefork error
Riccardo Magliocchetti [Fri, 1 Feb 2013 18:23:28 +0000 (19:23 +0100)]
afpd/fork: avoid use after free on of_closefork error

Reported by Coverity as CID #201311.