libpurple/stringref.c

Fri, 14 Feb 2020 01:49:34 -0600

author
Gary Kramlich <grim@reaperworld.com>
date
Fri, 14 Feb 2020 01:49:34 -0600
branch
use-after-free
changeset 40286
ca45e74fb1ee
parent 40141
ac2ec9dc4dc5
permissions
-rw-r--r--

closing merged branch

20147
66f05a854eee applied changes from 8a731bbd0197fbcc91a705c2d8f528154216defa
Richard Laager <rlaager@pidgin.im>
parents: 19859
diff changeset
1 /* purple
7763
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
2 *
15884
4de1981757fc sed -ie 's/gaim/purple/g'
Sean Egan <seanegan@pidgin.im>
parents: 15435
diff changeset
3 * Purple is the legal property of its developers, whose names are too numerous
8046
c581b20a47d6 [gaim-migrate @ 8730]
Sean Egan <seanegan@pidgin.im>
parents: 7796
diff changeset
4 * to list here. Please refer to the COPYRIGHT file distributed with this
c581b20a47d6 [gaim-migrate @ 8730]
Sean Egan <seanegan@pidgin.im>
parents: 7796
diff changeset
5 * source distribution.
7763
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
6 *
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
7 * This program is free software; you can redistribute it and/or modify
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
8 * it under the terms of the GNU General Public License as published by
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
9 * the Free Software Foundation; either version 2 of the License, or
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
10 * (at your option) any later version.
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
11 *
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
12 * This program is distributed in the hope that it will be useful,
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
13 * but WITHOUT ANY WARRANTY; without even the implied warranty of
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
14 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
15 * GNU General Public License for more details.
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
16 *
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
17 * You should have received a copy of the GNU General Public License
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
18 * along with this program; if not, write to the Free Software
19859
71d37b57eff2 The FSF changed its address a while ago; our files were out of date.
John Bailey <rekkanoryo@rekkanoryo.org>
parents: 15884
diff changeset
19 * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02111-1301 USA
7763
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
20 *
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
21 */
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
22
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
23 #include "internal.h"
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
24
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
25 #include <string.h>
7767
292e7cfea640 [gaim-migrate @ 8412]
Ethan Blanton <elb@pidgin.im>
parents: 7763
diff changeset
26 #include <stdarg.h>
7763
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
27
7789
2ab2604006e2 [gaim-migrate @ 8435]
Nathan Walp <nwalp@pidgin.im>
parents: 7788
diff changeset
28 #include "debug.h"
22351
6ca0640b3d31 Change some g_idle_add(...) calls in libpurple to purple_timeout_add(0, ...)
Mark Doliner <markdoliner@pidgin.im>
parents: 20147
diff changeset
29 #include "eventloop.h"
7763
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
30 #include "stringref.h"
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
31
35466
e259e75f1c6f Fix some gtk-doc warnings till xmlnode.h
Ankit Vani <a@nevitus.org>
parents: 35454
diff changeset
32 /*
14988
66b34458d49e [gaim-migrate @ 17698]
Richard Laager <rlaager@pidgin.im>
parents: 14254
diff changeset
33 * The internal representation of a stringref.
66b34458d49e [gaim-migrate @ 17698]
Richard Laager <rlaager@pidgin.im>
parents: 14254
diff changeset
34 *
66b34458d49e [gaim-migrate @ 17698]
Richard Laager <rlaager@pidgin.im>
parents: 14254
diff changeset
35 * @note For this structure to be useful, the string contained within
66b34458d49e [gaim-migrate @ 17698]
Richard Laager <rlaager@pidgin.im>
parents: 14254
diff changeset
36 * it must be immutable -- for this reason, do _not_ access it
66b34458d49e [gaim-migrate @ 17698]
Richard Laager <rlaager@pidgin.im>
parents: 14254
diff changeset
37 * directly!
66b34458d49e [gaim-migrate @ 17698]
Richard Laager <rlaager@pidgin.im>
parents: 14254
diff changeset
38 */
15884
4de1981757fc sed -ie 's/gaim/purple/g'
Sean Egan <seanegan@pidgin.im>
parents: 15435
diff changeset
39 struct _PurpleStringref {
35466
e259e75f1c6f Fix some gtk-doc warnings till xmlnode.h
Ankit Vani <a@nevitus.org>
parents: 35454
diff changeset
40 guint32 ref; /* The reference count of this string.
14988
66b34458d49e [gaim-migrate @ 17698]
Richard Laager <rlaager@pidgin.im>
parents: 14254
diff changeset
41 * Note that reference counts are only
66b34458d49e [gaim-migrate @ 17698]
Richard Laager <rlaager@pidgin.im>
parents: 14254
diff changeset
42 * 31 bits, and the high-order bit
66b34458d49e [gaim-migrate @ 17698]
Richard Laager <rlaager@pidgin.im>
parents: 14254
diff changeset
43 * indicates whether this string is up
66b34458d49e [gaim-migrate @ 17698]
Richard Laager <rlaager@pidgin.im>
parents: 14254
diff changeset
44 * for GC at the next idle handler...
66b34458d49e [gaim-migrate @ 17698]
Richard Laager <rlaager@pidgin.im>
parents: 14254
diff changeset
45 * But you aren't going to touch this
66b34458d49e [gaim-migrate @ 17698]
Richard Laager <rlaager@pidgin.im>
parents: 14254
diff changeset
46 * anyway, right? */
35466
e259e75f1c6f Fix some gtk-doc warnings till xmlnode.h
Ankit Vani <a@nevitus.org>
parents: 35454
diff changeset
47 char value[1]; /* The string contained in this ref.
14988
66b34458d49e [gaim-migrate @ 17698]
Richard Laager <rlaager@pidgin.im>
parents: 14254
diff changeset
48 * Notice that it is simply "hanging
66b34458d49e [gaim-migrate @ 17698]
Richard Laager <rlaager@pidgin.im>
parents: 14254
diff changeset
49 * off the end" of the ref ... this
66b34458d49e [gaim-migrate @ 17698]
Richard Laager <rlaager@pidgin.im>
parents: 14254
diff changeset
50 * is to save an allocation. */
66b34458d49e [gaim-migrate @ 17698]
Richard Laager <rlaager@pidgin.im>
parents: 14254
diff changeset
51 };
66b34458d49e [gaim-migrate @ 17698]
Richard Laager <rlaager@pidgin.im>
parents: 14254
diff changeset
52
7795
b0bb9c8c2795 [gaim-migrate @ 8441]
Ethan Blanton <elb@pidgin.im>
parents: 7790
diff changeset
53 #define REFCOUNT(x) ((x) & 0x7fffffff)
b0bb9c8c2795 [gaim-migrate @ 8441]
Ethan Blanton <elb@pidgin.im>
parents: 7790
diff changeset
54
7786
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
55 static GList *gclist = NULL;
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
56
15884
4de1981757fc sed -ie 's/gaim/purple/g'
Sean Egan <seanegan@pidgin.im>
parents: 15435
diff changeset
57 static void stringref_free(PurpleStringref *stringref);
7786
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
58 static gboolean gs_idle_cb(gpointer data);
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
59
15884
4de1981757fc sed -ie 's/gaim/purple/g'
Sean Egan <seanegan@pidgin.im>
parents: 15435
diff changeset
60 PurpleStringref *purple_stringref_new(const char *value)
7763
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
61 {
15884
4de1981757fc sed -ie 's/gaim/purple/g'
Sean Egan <seanegan@pidgin.im>
parents: 15435
diff changeset
62 PurpleStringref *newref;
32024
7d8e1af031fe Bounds-check stringref copies (which are safe anyway)
Ethan Blanton <elb@pidgin.im>
parents: 22351
diff changeset
63 size_t len;
7763
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
64
7772
6288282c84c1 [gaim-migrate @ 8417]
Ethan Blanton <elb@pidgin.im>
parents: 7768
diff changeset
65 if (value == NULL)
6288282c84c1 [gaim-migrate @ 8417]
Ethan Blanton <elb@pidgin.im>
parents: 7768
diff changeset
66 return NULL;
6288282c84c1 [gaim-migrate @ 8417]
Ethan Blanton <elb@pidgin.im>
parents: 7768
diff changeset
67
32024
7d8e1af031fe Bounds-check stringref copies (which are safe anyway)
Ethan Blanton <elb@pidgin.im>
parents: 22351
diff changeset
68 len = strlen(value);
7d8e1af031fe Bounds-check stringref copies (which are safe anyway)
Ethan Blanton <elb@pidgin.im>
parents: 22351
diff changeset
69
7d8e1af031fe Bounds-check stringref copies (which are safe anyway)
Ethan Blanton <elb@pidgin.im>
parents: 22351
diff changeset
70 newref = g_malloc(sizeof(PurpleStringref) + len);
33594
ef3d115642d7 Fix purple_stringref_new()
Richard Laager <rlaager@pidgin.im>
parents: 32024
diff changeset
71 /* g_strlcpy() takes the size of the buffer, including the NUL.
ef3d115642d7 Fix purple_stringref_new()
Richard Laager <rlaager@pidgin.im>
parents: 32024
diff changeset
72 strlen() returns the length of the string, without the NUL. */
ef3d115642d7 Fix purple_stringref_new()
Richard Laager <rlaager@pidgin.im>
parents: 32024
diff changeset
73 g_strlcpy(newref->value, value, len + 1);
7763
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
74 newref->ref = 1;
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
75
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
76 return newref;
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
77 }
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
78
15884
4de1981757fc sed -ie 's/gaim/purple/g'
Sean Egan <seanegan@pidgin.im>
parents: 15435
diff changeset
79 PurpleStringref *purple_stringref_new_noref(const char *value)
7786
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
80 {
15884
4de1981757fc sed -ie 's/gaim/purple/g'
Sean Egan <seanegan@pidgin.im>
parents: 15435
diff changeset
81 PurpleStringref *newref;
7786
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
82
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
83 if (value == NULL)
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
84 return NULL;
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
85
15884
4de1981757fc sed -ie 's/gaim/purple/g'
Sean Egan <seanegan@pidgin.im>
parents: 15435
diff changeset
86 newref = g_malloc(sizeof(PurpleStringref) + strlen(value));
7786
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
87 strcpy(newref->value, value);
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
88 newref->ref = 0x80000000;
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
89
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
90 if (gclist == NULL)
38433
361c801c4536 Remove purple_timeout_* function usage
Mike Ruprecht <cmaiku@gmail.com>
parents: 35466
diff changeset
91 g_timeout_add(0, gs_idle_cb, NULL);
7786
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
92 gclist = g_list_prepend(gclist, newref);
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
93
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
94 return newref;
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
95 }
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
96
15884
4de1981757fc sed -ie 's/gaim/purple/g'
Sean Egan <seanegan@pidgin.im>
parents: 15435
diff changeset
97 PurpleStringref *purple_stringref_printf(const char *format, ...)
7767
292e7cfea640 [gaim-migrate @ 8412]
Ethan Blanton <elb@pidgin.im>
parents: 7763
diff changeset
98 {
15884
4de1981757fc sed -ie 's/gaim/purple/g'
Sean Egan <seanegan@pidgin.im>
parents: 15435
diff changeset
99 PurpleStringref *newref;
7767
292e7cfea640 [gaim-migrate @ 8412]
Ethan Blanton <elb@pidgin.im>
parents: 7763
diff changeset
100 va_list ap;
292e7cfea640 [gaim-migrate @ 8412]
Ethan Blanton <elb@pidgin.im>
parents: 7763
diff changeset
101
292e7cfea640 [gaim-migrate @ 8412]
Ethan Blanton <elb@pidgin.im>
parents: 7763
diff changeset
102 if (format == NULL)
292e7cfea640 [gaim-migrate @ 8412]
Ethan Blanton <elb@pidgin.im>
parents: 7763
diff changeset
103 return NULL;
292e7cfea640 [gaim-migrate @ 8412]
Ethan Blanton <elb@pidgin.im>
parents: 7763
diff changeset
104
292e7cfea640 [gaim-migrate @ 8412]
Ethan Blanton <elb@pidgin.im>
parents: 7763
diff changeset
105 va_start(ap, format);
15884
4de1981757fc sed -ie 's/gaim/purple/g'
Sean Egan <seanegan@pidgin.im>
parents: 15435
diff changeset
106 newref = g_malloc(sizeof(PurpleStringref) + g_printf_string_upper_bound(format, ap));
7767
292e7cfea640 [gaim-migrate @ 8412]
Ethan Blanton <elb@pidgin.im>
parents: 7763
diff changeset
107 vsprintf(newref->value, format, ap);
292e7cfea640 [gaim-migrate @ 8412]
Ethan Blanton <elb@pidgin.im>
parents: 7763
diff changeset
108 va_end(ap);
7784
90779475b93b [gaim-migrate @ 8429]
Ethan Blanton <elb@pidgin.im>
parents: 7772
diff changeset
109 newref->ref = 1;
7767
292e7cfea640 [gaim-migrate @ 8412]
Ethan Blanton <elb@pidgin.im>
parents: 7763
diff changeset
110
292e7cfea640 [gaim-migrate @ 8412]
Ethan Blanton <elb@pidgin.im>
parents: 7763
diff changeset
111 return newref;
292e7cfea640 [gaim-migrate @ 8412]
Ethan Blanton <elb@pidgin.im>
parents: 7763
diff changeset
112 }
292e7cfea640 [gaim-migrate @ 8412]
Ethan Blanton <elb@pidgin.im>
parents: 7763
diff changeset
113
15884
4de1981757fc sed -ie 's/gaim/purple/g'
Sean Egan <seanegan@pidgin.im>
parents: 15435
diff changeset
114 PurpleStringref *purple_stringref_ref(PurpleStringref *stringref)
7763
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
115 {
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
116 if (stringref == NULL)
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
117 return NULL;
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
118 stringref->ref++;
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
119 return stringref;
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
120 }
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
121
15884
4de1981757fc sed -ie 's/gaim/purple/g'
Sean Egan <seanegan@pidgin.im>
parents: 15435
diff changeset
122 void purple_stringref_unref(PurpleStringref *stringref)
7763
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
123 {
7790
e8943c3082f7 [gaim-migrate @ 8436]
Ethan Blanton <elb@pidgin.im>
parents: 7789
diff changeset
124 if (stringref == NULL)
e8943c3082f7 [gaim-migrate @ 8436]
Ethan Blanton <elb@pidgin.im>
parents: 7789
diff changeset
125 return;
7795
b0bb9c8c2795 [gaim-migrate @ 8441]
Ethan Blanton <elb@pidgin.im>
parents: 7790
diff changeset
126 if (REFCOUNT(--(stringref->ref)) == 0) {
7786
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
127 if (stringref->ref & 0x80000000)
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
128 gclist = g_list_remove(gclist, stringref);
7788
8bcade83dca9 [gaim-migrate @ 8434]
Ethan Blanton <elb@pidgin.im>
parents: 7786
diff changeset
129 stringref_free(stringref);
7786
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
130 }
7763
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
131 }
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
132
15884
4de1981757fc sed -ie 's/gaim/purple/g'
Sean Egan <seanegan@pidgin.im>
parents: 15435
diff changeset
133 const char *purple_stringref_value(const PurpleStringref *stringref)
7763
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
134 {
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
135 return (stringref == NULL ? NULL : stringref->value);
0a0c2c3acf78 [gaim-migrate @ 8408]
Ethan Blanton <elb@pidgin.im>
parents:
diff changeset
136 }
7786
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
137
15884
4de1981757fc sed -ie 's/gaim/purple/g'
Sean Egan <seanegan@pidgin.im>
parents: 15435
diff changeset
138 int purple_stringref_cmp(const PurpleStringref *s1, const PurpleStringref *s2)
7786
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
139 {
15884
4de1981757fc sed -ie 's/gaim/purple/g'
Sean Egan <seanegan@pidgin.im>
parents: 15435
diff changeset
140 return (s1 == s2 ? 0 : strcmp(purple_stringref_value(s1), purple_stringref_value(s2)));
7786
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
141 }
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
142
15884
4de1981757fc sed -ie 's/gaim/purple/g'
Sean Egan <seanegan@pidgin.im>
parents: 15435
diff changeset
143 size_t purple_stringref_len(const PurpleStringref *stringref)
7786
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
144 {
15884
4de1981757fc sed -ie 's/gaim/purple/g'
Sean Egan <seanegan@pidgin.im>
parents: 15435
diff changeset
145 return strlen(purple_stringref_value(stringref));
7786
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
146 }
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
147
15884
4de1981757fc sed -ie 's/gaim/purple/g'
Sean Egan <seanegan@pidgin.im>
parents: 15435
diff changeset
148 static void stringref_free(PurpleStringref *stringref)
7786
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
149 {
40139
13cfb1168f42 Add NULL checks
qarkai <qarkai@gmail.com>
parents: 40135
diff changeset
150 g_return_if_fail(stringref != NULL);
13cfb1168f42 Add NULL checks
qarkai <qarkai@gmail.com>
parents: 40135
diff changeset
151
40135
3acbe4afeab5 Reduce lists iterations
qarkai <qarkai@gmail.com>
parents: 38433
diff changeset
152 if (REFCOUNT(stringref->ref) == 0) {
3acbe4afeab5 Reduce lists iterations
qarkai <qarkai@gmail.com>
parents: 38433
diff changeset
153 g_free(stringref);
7786
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
154 return;
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
155 }
40135
3acbe4afeab5 Reduce lists iterations
qarkai <qarkai@gmail.com>
parents: 38433
diff changeset
156 #ifdef DEBUG
3acbe4afeab5 Reduce lists iterations
qarkai <qarkai@gmail.com>
parents: 38433
diff changeset
157 purple_debug(PURPLE_DEBUG_ERROR, "stringref", "Free of nonzero (%d) ref stringref!\n", REFCOUNT(stringref->ref));
7786
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
158 #endif /* DEBUG */
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
159 }
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
160
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
161 static gboolean gs_idle_cb(gpointer data)
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
162 {
40135
3acbe4afeab5 Reduce lists iterations
qarkai <qarkai@gmail.com>
parents: 38433
diff changeset
163 g_list_free_full(gclist, (GDestroyNotify)stringref_free);
40141
ac2ec9dc4dc5 Set GList to NULL after free
qarkai <qarkai@gmail.com>
parents: 40139
diff changeset
164 gclist = NULL;
7786
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
165
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
166 return FALSE;
bcc5728e8810 [gaim-migrate @ 8431]
Ethan Blanton <elb@pidgin.im>
parents: 7784
diff changeset
167 }

mercurial