Mercurial > pidgin
annotate Makefile.am @ 15976:a6a79b8616bf
I commonly see a crash in which socket_ready_cb(), shortly after a laptop wakes from sleep, is passed invalid (previously freed) connect_data. It looks like this:
Thread 0 Crashed:
0 libobjc.A.dylib 0x90a59380 objc_msgSend 16
1 Libgaim 0x0fe23bcd gaim_proxy_connect_data_disconnect 172
2 Libgaim 0x0fe23d63 socket_ready_cb 199
3 com.apple.CoreFoundation 0x90843ffd __CFSocketDoCallback 551
(objc_msgSend is how ObjC routes messages... it's being called because connect_data->cconnect_cb is invalid).
It appears that when this crash happens, the socket is marked as ready just before the computer sleeps; on the next run loop, the callback will be called [socket_ready_cb()]. The computer sleeps and every account is disconnected first, which calls gaim_proxy_connect_cancel_with_handle(), destroying the connect_data. When it awakens, it calls socket_ready_cb() and the crash occurs.
I've added PURPLE_PROXY_CONNECT_DATA_IS_VALID, which takes advantage of the fact that all valid connect_data objects are stored in the handles GSList, just as PURPLE_GAIM_CONNECTION_IS_VALID works.
author | Evan Schoenberg <evan.s@dreskin.net> |
---|---|
date | Sun, 01 Apr 2007 02:17:06 +0000 |
parents | 88d626a179a6 |
children | 1260354ed92d |
rev | line source |
---|---|
15603 | 1 EXTRA_DIST = \ |
7960 | 2 COPYRIGHT \ |
14506 | 3 ChangeLog.API \ |
4 ChangeLog.win32 \ | |
6198
fcc28f5dd0c8
[gaim-migrate @ 6684]
Herman Bloggs <hermanator12002@yahoo.com>
parents:
6187
diff
changeset
|
5 Doxyfile.in \ |
14506 | 6 HACKING \ |
7 Makefile.mingw \ | |
8 PLUGIN_HOWTO \ | |
9 PROGRAMMING_NOTES \ | |
10 README.SVN \ | |
11 README.dbus \ | |
12 README.mingw \ | |
15903
6f04fe174f40
Change some stuff from gaim to pidgin
Mark Doliner <mark@kingant.net>
parents:
15887
diff
changeset
|
13 config.h.mingw \ |
9364 | 14 gaim.pc.in \ |
15884
c6e563dfaa7a
More s/gaim/pidgin/ and s/gaim/purple/ and make the RPM spec file work a bit
Stu Tomlinson <stu@nosnilmot.com>
parents:
15817
diff
changeset
|
15 gaim-uninstalled.pc.in \ |
c6e563dfaa7a
More s/gaim/pidgin/ and s/gaim/purple/ and make the RPM spec file work a bit
Stu Tomlinson <stu@nosnilmot.com>
parents:
15817
diff
changeset
|
16 gaim.service.in \ |
13179
5adc0c9da9f3
[gaim-migrate @ 15542]
Richard Laager <rlaager@wiktel.com>
parents:
12299
diff
changeset
|
17 intltool-extract.in \ |
5adc0c9da9f3
[gaim-migrate @ 15542]
Richard Laager <rlaager@wiktel.com>
parents:
12299
diff
changeset
|
18 intltool-merge.in \ |
5adc0c9da9f3
[gaim-migrate @ 15542]
Richard Laager <rlaager@wiktel.com>
parents:
12299
diff
changeset
|
19 intltool-update.in \ |
15903
6f04fe174f40
Change some stuff from gaim to pidgin
Mark Doliner <mark@kingant.net>
parents:
15887
diff
changeset
|
20 pidgin.apspec.in \ |
6f04fe174f40
Change some stuff from gaim to pidgin
Mark Doliner <mark@kingant.net>
parents:
15887
diff
changeset
|
21 pidgin.spec.in \ |
6f04fe174f40
Change some stuff from gaim to pidgin
Mark Doliner <mark@kingant.net>
parents:
15887
diff
changeset
|
22 pidgin.desktop.in \ |
6198
fcc28f5dd0c8
[gaim-migrate @ 6684]
Herman Bloggs <hermanator12002@yahoo.com>
parents:
6187
diff
changeset
|
23 po/Makefile.mingw |
0 | 24 |
14652 | 25 noinst_HEADERS = config.h |
9364 | 26 |
13490
40c5d0508a1a
[gaim-migrate @ 15866]
Gary Kramlich <grim@reaperworld.com>
parents:
13318
diff
changeset
|
27 if ENABLE_DBUS |
40c5d0508a1a
[gaim-migrate @ 15866]
Gary Kramlich <grim@reaperworld.com>
parents:
13318
diff
changeset
|
28 dbus_servicedir=$(DBUS_SERVICES_DIR) |
40c5d0508a1a
[gaim-migrate @ 15866]
Gary Kramlich <grim@reaperworld.com>
parents:
13318
diff
changeset
|
29 dbus_service_DATA=gaim.service |
40c5d0508a1a
[gaim-migrate @ 15866]
Gary Kramlich <grim@reaperworld.com>
parents:
13318
diff
changeset
|
30 endif |
40c5d0508a1a
[gaim-migrate @ 15866]
Gary Kramlich <grim@reaperworld.com>
parents:
13318
diff
changeset
|
31 |
15884
c6e563dfaa7a
More s/gaim/pidgin/ and s/gaim/purple/ and make the RPM spec file work a bit
Stu Tomlinson <stu@nosnilmot.com>
parents:
15817
diff
changeset
|
32 dist-hook: pidgin.spec |
c6e563dfaa7a
More s/gaim/pidgin/ and s/gaim/purple/ and make the RPM spec file work a bit
Stu Tomlinson <stu@nosnilmot.com>
parents:
15817
diff
changeset
|
33 cp pidgin.spec $(distdir) |
14750
2a7700895952
[gaim-migrate @ 17507]
Daniel Atallah <daniel.atallah@gmail.com>
parents:
14652
diff
changeset
|
34 rm $(distdir)/config.h |
3449
8c6950b6eb82
[gaim-migrate @ 3493]
Christian Hammond <chipx86@chipx86.com>
parents:
3439
diff
changeset
|
35 |
15887
8a2e7b406b4e
One step towards passing "make distcheck"
Stu Tomlinson <stu@nosnilmot.com>
parents:
15884
diff
changeset
|
36 distcheck-hook: libpurple/plugins/perl/common/Purple.pm pidgin/plugins/perl/common/GtkUI.pm |
15373
5fe8042783c1
Rename gtk/ and libgaim/ to pidgin/ and libpurple/
Sean Egan <seanegan@gmail.com>
parents:
14794
diff
changeset
|
37 # cp libpurple/plugins/perl/common/Gaim.pm $(distdir)/libpurple/plugins/perl/common |
6701
b7e113a59b51
[gaim-migrate @ 7227]
Christian Hammond <chipx86@chipx86.com>
parents:
6596
diff
changeset
|
38 |
5253
2c9de5ae24a0
[gaim-migrate @ 5625]
Christian Hammond <chipx86@chipx86.com>
parents:
4564
diff
changeset
|
39 appsdir = $(datadir)/applications |
15478 | 40 apps_in_files = pidgin.desktop.in |
13179
5adc0c9da9f3
[gaim-migrate @ 15542]
Richard Laager <rlaager@wiktel.com>
parents:
12299
diff
changeset
|
41 apps_DATA = $(apps_in_files:.desktop.in=.desktop) |
5adc0c9da9f3
[gaim-migrate @ 15542]
Richard Laager <rlaager@wiktel.com>
parents:
12299
diff
changeset
|
42 @INTLTOOL_DESKTOP_RULE@ |
327 | 43 |
14191
009db0b357b5
This is a hand-crafted commit to migrate across subversion revisions
Ethan Blanton <elb@pidgin.im>
parents:
13490
diff
changeset
|
44 if ENABLE_GTK |
15373
5fe8042783c1
Rename gtk/ and libgaim/ to pidgin/ and libpurple/
Sean Egan <seanegan@gmail.com>
parents:
14794
diff
changeset
|
45 GTK_DIR=pidgin |
14191
009db0b357b5
This is a hand-crafted commit to migrate across subversion revisions
Ethan Blanton <elb@pidgin.im>
parents:
13490
diff
changeset
|
46 endif |
009db0b357b5
This is a hand-crafted commit to migrate across subversion revisions
Ethan Blanton <elb@pidgin.im>
parents:
13490
diff
changeset
|
47 |
009db0b357b5
This is a hand-crafted commit to migrate across subversion revisions
Ethan Blanton <elb@pidgin.im>
parents:
13490
diff
changeset
|
48 if ENABLE_GNT |
15817
0e3a8505ebbe
renamed gaim-text to finch
Sean Egan <seanegan@gmail.com>
parents:
15751
diff
changeset
|
49 GNT_DIR=finch |
14191
009db0b357b5
This is a hand-crafted commit to migrate across subversion revisions
Ethan Blanton <elb@pidgin.im>
parents:
13490
diff
changeset
|
50 endif |
009db0b357b5
This is a hand-crafted commit to migrate across subversion revisions
Ethan Blanton <elb@pidgin.im>
parents:
13490
diff
changeset
|
51 |
15373
5fe8042783c1
Rename gtk/ and libgaim/ to pidgin/ and libpurple/
Sean Egan <seanegan@gmail.com>
parents:
14794
diff
changeset
|
52 SUBDIRS = libpurple doc $(GNT_DIR) $(GTK_DIR) m4macros po |
4554
fbf9f873abfc
[gaim-migrate @ 4834]
Christian Hammond <chipx86@chipx86.com>
parents:
3726
diff
changeset
|
53 |
6596
dcb336f3d9a6
[gaim-migrate @ 7120]
Christian Hammond <chipx86@chipx86.com>
parents:
6411
diff
changeset
|
54 docs: Doxyfile |
10925
993db24dae16
[gaim-migrate @ 12696]
Gary Kramlich <grim@reaperworld.com>
parents:
10362
diff
changeset
|
55 if HAVE_DOXYGEN |
993db24dae16
[gaim-migrate @ 12696]
Gary Kramlich <grim@reaperworld.com>
parents:
10362
diff
changeset
|
56 @echo "Running doxygen..." |
4554
fbf9f873abfc
[gaim-migrate @ 4834]
Christian Hammond <chipx86@chipx86.com>
parents:
3726
diff
changeset
|
57 @doxygen |
10925
993db24dae16
[gaim-migrate @ 12696]
Gary Kramlich <grim@reaperworld.com>
parents:
10362
diff
changeset
|
58 else |
993db24dae16
[gaim-migrate @ 12696]
Gary Kramlich <grim@reaperworld.com>
parents:
10362
diff
changeset
|
59 @echo "doxygen was not found during configure. Aborting." |
993db24dae16
[gaim-migrate @ 12696]
Gary Kramlich <grim@reaperworld.com>
parents:
10362
diff
changeset
|
60 @echo; |
993db24dae16
[gaim-migrate @ 12696]
Gary Kramlich <grim@reaperworld.com>
parents:
10362
diff
changeset
|
61 endif |
11862 | 62 |
63 # perl's MakeMaker uninstall foo doesn't work well with DESTDIR set, which | |
64 # breaks "make distcheck" unless we ignore perl things | |
65 | |
66 distuninstallcheck_listfiles = \ | |
15887
8a2e7b406b4e
One step towards passing "make distcheck"
Stu Tomlinson <stu@nosnilmot.com>
parents:
15884
diff
changeset
|
67 find . -type f -print | grep -v perl | grep -v Purple.3pm |
12299 | 68 |
15910
88d626a179a6
Rename our schemas file and update the url handler to point to the
Mark Doliner <mark@kingant.net>
parents:
15903
diff
changeset
|
69 DISTCLEANFILES= pidgin.desktop libpurple/gconf/purple.schemas intltool-extract \ |
14769
61aa08fcd315
[gaim-migrate @ 17527]
Luke Schierer <lschiere@pidgin.im>
parents:
14764
diff
changeset
|
70 intltool-merge intltool-update |
15400
74ab783e3676
use available icon in statusbox
Sean Egan <seanegan@gmail.com>
parents:
15373
diff
changeset
|
71 |
74ab783e3676
use available icon in statusbox
Sean Egan <seanegan@gmail.com>
parents:
15373
diff
changeset
|
72 ACLOCAL_AMFLAGS = -I m4 |