summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJeremy Allison <jra@samba.org>2009-09-03 07:38:21 -0700
committerJeremy Allison <jra@samba.org>2009-09-03 07:38:21 -0700
commitc97698e762b1ea8d7133f04ae822225676a6f135 (patch)
tree120352a0a3c3a8f04fb726652f06a50ae3ec47d3
parent38995d7cbb0c3143eb1fea0215863321349c7bbe (diff)
downloadsamba-c97698e762b1ea8d7133f04ae822225676a6f135.tar.gz
samba-c97698e762b1ea8d7133f04ae822225676a6f135.tar.bz2
samba-c97698e762b1ea8d7133f04ae822225676a6f135.zip
Another part of the fix for bug 6651 - smbd SIGSEGV when breaking oplocks.
SA_INFO_QUEUE_COUNT *MUST* be a power of 2, in order for the ring buffer wrap to work correctly at the 32 bit boundary. Thanks to Petr Vandrovec <petr@vandrovec.name> for this.
-rw-r--r--lib/tevent/tevent_signal.c8
1 files changed, 6 insertions, 2 deletions
diff --git a/lib/tevent/tevent_signal.c b/lib/tevent/tevent_signal.c
index b329f8c1e7..ef9c0cf288 100644
--- a/lib/tevent/tevent_signal.c
+++ b/lib/tevent/tevent_signal.c
@@ -32,8 +32,12 @@
#define NUM_SIGNALS 64
-/* maximum number of SA_SIGINFO signals to hold in the queue */
-#define SA_INFO_QUEUE_COUNT 100
+/* maximum number of SA_SIGINFO signals to hold in the queue.
+ NB. This *MUST* be a power of 2, in order for the ring buffer
+ wrap to work correctly. Thanks to Petr Vandrovec <petr@vandrovec.name>
+ for this. */
+
+#define SA_INFO_QUEUE_COUNT 64
struct sigcounter {
uint32_t count;