diff options
author | Guy Harris <guy@alum.mit.edu> | 2014-07-15 16:40:46 -0700 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2014-07-15 23:43:32 +0000 |
commit | d4dab16a3fed4dc399083e454226675894b42871 (patch) | |
tree | 24be814f0f9d36acf67a4df4bc59ae11e8fc9576 /wsutil/buffer.h | |
parent | 61ac8156812f5f9ef277f303a5e22aac375ad357 (diff) | |
download | wireshark-d4dab16a3fed4dc399083e454226675894b42871.tar.gz |
Only one buffer.c, please.
Otherwise, if you link with both libwiretap and libfiletap, it's
anybody's guess which one you get. That means you're wasting memory
with two copies of its routines if they're identical, and means
surprising behavior if they're not (which showed up when I was debugging
a double-free crash - fixing libwiretap's buffer_free() didn't fix the
problem, because Wireshark happened to be calling libfiletap' unfixed
buffer_free()).
There's nothing *tap-specific about Buffers, anyway, so it really
belongs in wsutil.
Change-Id: I91537e46917e91277981f8f3365a2c0873152870
Reviewed-on: https://code.wireshark.org/review/3066
Reviewed-by: Guy Harris <guy@alum.mit.edu>
Diffstat (limited to 'wsutil/buffer.h')
-rw-r--r-- | wsutil/buffer.h | 72 |
1 files changed, 72 insertions, 0 deletions
diff --git a/wsutil/buffer.h b/wsutil/buffer.h new file mode 100644 index 0000000000..774dcdddb6 --- /dev/null +++ b/wsutil/buffer.h @@ -0,0 +1,72 @@ +/* buffer.h + * + * Wiretap Library + * Copyright (c) 1998 by Gilbert Ramirez <gram@alumni.rice.edu> + * + * 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. + * + */ + +#ifndef __W_BUFFER_H__ +#define __W_BUFFER_H__ + +#include <glib.h> +#include "ws_symbol_export.h" + +#ifdef __cplusplus +extern "C" { +#endif /* __cplusplus */ + +#define SOME_FUNCTIONS_ARE_DEFINES + +typedef struct Buffer { + guint8 *data; + gsize allocated; + gsize start; + gsize first_free; +} Buffer; + +WS_DLL_PUBLIC +void buffer_init(Buffer* buffer, gsize space); +WS_DLL_PUBLIC +void buffer_free(Buffer* buffer); +WS_DLL_PUBLIC +void buffer_assure_space(Buffer* buffer, gsize space); +WS_DLL_PUBLIC +void buffer_append(Buffer* buffer, guint8 *from, gsize bytes); +WS_DLL_PUBLIC +void buffer_remove_start(Buffer* buffer, gsize bytes); + +#ifdef SOME_FUNCTIONS_ARE_DEFINES +# define buffer_clean(buffer) buffer_remove_start((buffer), buffer_length(buffer)) +# define buffer_increase_length(buffer,bytes) (buffer)->first_free += (bytes) +# define buffer_length(buffer) ((buffer)->first_free - (buffer)->start) +# define buffer_start_ptr(buffer) ((buffer)->data + (buffer)->start) +# define buffer_end_ptr(buffer) ((buffer)->data + (buffer)->first_free) +# define buffer_append_buffer(buffer,src_buffer) buffer_append((buffer), buffer_start_ptr(src_buffer), buffer_length(src_buffer)) +#else + void buffer_clean(Buffer* buffer); + void buffer_increase_length(Buffer* buffer, unsigned int bytes); + unsigned int buffer_length(Buffer* buffer); + guint8* buffer_start_ptr(Buffer* buffer); + guint8* buffer_end_ptr(Buffer* buffer); + void buffer_append_buffer(Buffer* buffer, Buffer* src_buffer); +#endif + +#ifdef __cplusplus +} +#endif /* __cplusplus */ + +#endif |