summaryrefslogtreecommitdiff
path: root/buildtools/wafadmin
diff options
context:
space:
mode:
authorVolker Lendecke <vl@samba.org>2012-03-31 13:37:20 +0200
committerVolker Lendecke <vl@samba.org>2012-03-31 15:25:54 +0200
commit0aacdbfada46329e0ad9dacfa90041a1c7dbf3e8 (patch)
tree0146d41f2de03331c80a19c671745efa6e98381d /buildtools/wafadmin
parent7f7c2d721d1e336a86b29fecc81f5c0e28d105dc (diff)
downloadsamba-0aacdbfada46329e0ad9dacfa90041a1c7dbf3e8.tar.gz
samba-0aacdbfada46329e0ad9dacfa90041a1c7dbf3e8.tar.bz2
samba-0aacdbfada46329e0ad9dacfa90041a1c7dbf3e8.zip
s3-aio-fork: Fix a segfault in vfs_aio_fork
aio_suspend does not signal the main process with a signal, it just waits. The aio_fork module does not use the signal at all, it directly calls back into the main smbd by calling smbd_aio_complete_aio_ex. This is an abstraction violation, but the alternative would have been to use signals where they are not needed. However, in wait_for_aio_completion this bites us: With aio_fork we call handle_aio_completed twice on the same aio_ex struct: Once from the call to handle_aio_completion within the aio_fork module and once from the code in wait_for_aio_completion. This patch fixes it in a pretty bad way by introducing flag variables and more state. But the mid-term plan is to replace the posix aio calls from the vfs and do pread_send/recv and pwrite_send/recv at the vfs layer, so this will significantly change anyway. Thanks to Kirill Malkin <kirill.malkin@starboardstorage.com> for reporting this crash! Autobuild-User: Volker Lendecke <vl@samba.org> Autobuild-Date: Sat Mar 31 15:25:55 CEST 2012 on sn-devel-104
Diffstat (limited to 'buildtools/wafadmin')
0 files changed, 0 insertions, 0 deletions