Mercurial > pidgin.yaz
annotate doc/core-signals.dox @ 23995:85bed17fe5c1
The variable we use to keep track of the watcher of the ssl connection
should be unsigned. This isn't really a problem in Pidgin, where we
use glib's mainloop and GIOChannels because glib starts assigning the
handle IDs sequentially starting from 1.
But if an eventloop implementation ever returns a handle ID greater
than the largest possible signed integer (2,147,483,647) then we
won't be able to remove the watcher because purple_ssl_close() in
sslconn.c only removes it if inpa > 0, and since it interprets inpa
as a signed value then handles over 2,147,483,647 appear as negative
numbers.
I stumbled upon this when playing around with libevent, which can
use epoll. My implementation generated a random handle ID which
was sometimes greater than 2,147,483,647.
I don't believe this breaks binary compatibility. And I don't think
it breaks source compatibility, but I guess it might depend on what
compiler you're using.
author | Mark Doliner <mark@kingant.net> |
---|---|
date | Thu, 04 Sep 2008 18:04:29 +0000 |
parents | e0613cf8c493 |
children | 4179ab2cfe1f |
rev | line source |
---|---|
12053 | 1 /** @page core-signals Core Signals |
2 | |
3 @signals | |
4 @signal quitting | |
5 @endsignals | |
6 | |
20807
e0613cf8c493
Add some links from signal documentation back to the documentation for the
Will Thompson <will.thompson@collabora.co.uk>
parents:
16196
diff
changeset
|
7 @see core.h |
e0613cf8c493
Add some links from signal documentation back to the documentation for the
Will Thompson <will.thompson@collabora.co.uk>
parents:
16196
diff
changeset
|
8 |
12053 | 9 <hr> |
10 | |
11 @signaldef quitting | |
12 @signalproto | |
13 void (*quitting)(); | |
14 @endsignalproto | |
15 @signaldesc | |
16196 | 16 Emitted when libpurple is quitting. |
12053 | 17 @endsignaldef |
18 | |
19 */ | |
20807
e0613cf8c493
Add some links from signal documentation back to the documentation for the
Will Thompson <will.thompson@collabora.co.uk>
parents:
16196
diff
changeset
|
20 // vim: syntax=c.doxygen tw=75 et |