diff options
author | Mikael Kanstrup <mikael.kanstrup@gmail.com> | 2017-08-25 14:05:42 +0200 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2017-08-25 19:41:34 +0000 |
commit | b1a041898333a8d38387dca4fa6d509881845072 (patch) | |
tree | 3eb0abd246fae05d218e7e4a3e06fb6f875814a4 /caputils/capture_ifinfo.h | |
parent | 9ae80aea082384605e647f55fbc4d2910811a929 (diff) |
Fix leaked timestamp records
Valgrind reports leaked timestamp records.
A comment stated that the timestamp info members only contain
static data. That claim was only true for some cases, not all so
make all cases allocate memory and have them properly freed when
removed.
Fixes: aca55a2 ("Add hardware timestamping support")
Change-Id: I31e4689070019ad1f531008394e7d6e48318c70c
Reviewed-on: https://code.wireshark.org/review/23206
Petri-Dish: Anders Broman <a.broman58@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Guy Harris <guy@alum.mit.edu>
Diffstat (limited to 'caputils/capture_ifinfo.h')
-rw-r--r-- | caputils/capture_ifinfo.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/caputils/capture_ifinfo.h b/caputils/capture_ifinfo.h index 48c54feec8..65084b8d60 100644 --- a/caputils/capture_ifinfo.h +++ b/caputils/capture_ifinfo.h @@ -115,8 +115,8 @@ typedef struct { * Information about timestamp types. */ typedef struct { - const char *name; /* e.g. "adapter_unsynced" */ - const char *description; /* description from libpcap e.g. "Adapter, not synced with system time" */ + char *name; /* e.g. "adapter_unsynced" */ + char *description; /* description from libpcap e.g. "Adapter, not synced with system time" */ } timestamp_info_t; /** |