1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
|
/* wmem_scopes.c
* Wireshark Memory Manager Scopes
* Copyright 2012, Evan Huus <eapache@gmail.com>
*
* $Id$
*
* Wireshark - Network traffic analyzer
* By Gerald Combs <gerald@wireshark.org>
* Copyright 1998 Gerald Combs
*
* This program is free software; you can redistribute it and/or modify
* it under the terms of the GNU General Public License as published by
* the Free Software Foundation; either version 2 of the License, or
* (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License along
* with this program; if not, write to the Free Software Foundation, Inc.,
* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
*/
#include <glib.h>
#include "wmem_scopes.h"
#include "wmem_allocator_glib.h"
#include "wmem_allocator_block.h"
/* One of the supposed benefits of wmem over the old emem was going to be that
* the scoping of the various memory pools would be obvious, since they would
* no longer be global. Instead, the pools would be managed as variables scoped
* by the compiler, so functions outside of that scope wouldn't have access to
* the pools and wouldn't be able to allocate memory in a scope to which they
* didn't belong.
*
* That idea fell apart rather quickly :P
*
* The principle still stands, and most pools should be managed in that way.
* The three in this file are *exceptions*. They are the three scopes that emem
* provided as globals. Converting all of the code that used them to pass an
* extra parameter (or three) around would have been a nightmare of epic
* proportions, so we provide these three as globals still.
*
* We do, however, use some extra booleans and a mountain of assertions to try
* and catch anybody accessing the pools out of the correct scope. It's not
* perfect, but it should stop most of the bad behaviour that emem permitted.
*/
/* TODO: Make these thread-local */
static wmem_allocator_t *packet_scope = NULL;
static wmem_allocator_t *file_scope = NULL;
static wmem_allocator_t *epan_scope = NULL;
static gboolean in_packet_scope = FALSE;
static gboolean in_file_scope = FALSE;
/* Packet Scope */
wmem_allocator_t *
wmem_packet_scope(void)
{
g_assert(packet_scope);
g_assert(in_packet_scope);
return epan_scope;
}
void
wmem_enter_packet_scope(void)
{
g_assert(packet_scope);
g_assert(in_file_scope);
g_assert(!in_packet_scope);
in_packet_scope = TRUE;
}
void
wmem_leave_packet_scope(void)
{
g_assert(packet_scope);
g_assert(in_packet_scope);
wmem_free_all(packet_scope);
in_packet_scope = FALSE;
}
/* File Scope */
wmem_allocator_t *
wmem_file_scope(void)
{
g_assert(file_scope);
g_assert(in_file_scope);
return epan_scope;
}
void
wmem_enter_file_scope(void)
{
g_assert(file_scope);
g_assert(!in_file_scope);
in_file_scope = TRUE;
}
void
wmem_leave_file_scope(void)
{
/* XXX: cleanup_dissection (the current caller of this function) is
* itself sometimes called when no file exists to be cleaned up. It's not
* a huge problem really, but it means that we can't assert in_file_scope
* here because it's not always true.
*
* At some point the code should be fixed so that cleanup_dissection is
* only ever called when it's really needed.
*
* g_assert(in_file_scope);
*/
g_assert(file_scope);
g_assert(!in_packet_scope);
wmem_free_all(file_scope);
in_file_scope = FALSE;
}
/* Epan Scope */
wmem_allocator_t *
wmem_epan_scope(void)
{
g_assert(epan_scope);
return epan_scope;
}
/* Scope Management */
void
wmem_init_scopes(void)
{
g_assert(packet_scope == NULL);
g_assert(file_scope == NULL);
g_assert(epan_scope == NULL);
g_assert(in_packet_scope == FALSE);
g_assert(in_file_scope == FALSE);
packet_scope = wmem_create_block_allocator();
file_scope = wmem_create_glib_allocator();
epan_scope = wmem_create_glib_allocator();
}
void
wmem_cleanup_scopes(void)
{
g_assert(packet_scope);
g_assert(file_scope);
g_assert(epan_scope);
g_assert(in_packet_scope == FALSE);
g_assert(in_file_scope == FALSE);
wmem_destroy_allocator(packet_scope);
wmem_destroy_allocator(file_scope);
wmem_destroy_allocator(epan_scope);
packet_scope = NULL;
file_scope = NULL;
epan_scope = NULL;
}
/*
* Editor modelines - http://www.wireshark.org/tools/modelines.html
*
* Local variables:
* c-basic-offset: 4
* tab-width: 8
* indent-tabs-mode: nil
* End:
*
* vi: set shiftwidth=4 tabstop=8 expandtab:
* :indentSize=4:tabSize=8:noTabs=true:
*/
|
