unixsock plugin: Fix a (well hidden) race condition.
authorFlorian Forster <octo@huhu.verplant.org>
Wed, 30 Sep 2009 20:49:16 +0000 (22:49 +0200)
committerFlorian Forster <octo@huhu.verplant.org>
Wed, 30 Sep 2009 20:49:16 +0000 (22:49 +0200)
commit5b5fff5bbdc5515d7f4c2a01df47373f4ac44847
tree79444ec02839e6e191e838f8ccadf5be3cfabe3a
parentfdba679afa5681557916095e393adcbecacbdc61
unixsock plugin: Fix a (well hidden) race condition.

Within the client handling thread, fdopen is called twice on the file
descriptor passed to the thread. Later those file handles are closed like:

  fclose (fhin);
  fclose (fhout);

This is a race condition, because the first call to fclose will close the file
descriptor. The second call to fclose will try the same. Usually, it would fail
silently and all is well. On a busy machine, however, another thread may just
have opened a file or accepted a socket. In that case an arbitrary file
descriptor is closed. If the file descriptor is opened yet again fast enough,
data may even end up in a totally wrong location.

As a work-around the file descriptor is not dup'ed so each fdopen operates on
its own file descriptor. As an alternative the "r+" mode and a single file
handle may be suitable, too.

Many thanks to Sven Trenkel for pointing me into the right directioin :)
src/unixsock.c