diff options
| author | morriss <morriss@f5534014-38df-0310-8fa8-9805f1628bb7> | 2010-05-17 21:49:09 +0000 |
|---|---|---|
| committer | morriss <morriss@f5534014-38df-0310-8fa8-9805f1628bb7> | 2010-05-17 21:49:09 +0000 |
| commit | 484d8e17fd6e007427d6f76bfb326307ab2d3d30 (patch) | |
| tree | 01b3ba1b28efff6b5a125a129589148a38ceb053 /epan/dissectors/packet-smb-pipe.c | |
| parent | 51c0d9957368cea32bb73ba856068b6212d7672b (diff) | |
Don't pass a NULL pointer to what is supposed to be (according to the format) a string: some implementations of vsnprintf() will core on that. (Unfortunately it seems that g_vsnprintf() doesn't protect us from this...)
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@32848 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'epan/dissectors/packet-smb-pipe.c')
| -rw-r--r-- | epan/dissectors/packet-smb-pipe.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-smb-pipe.c b/epan/dissectors/packet-smb-pipe.c index 118ac0b439..4bda305604 100644 --- a/epan/dissectors/packet-smb-pipe.c +++ b/epan/dissectors/packet-smb-pipe.c @@ -2138,7 +2138,7 @@ dissect_transact_data(tvbuff_t *tvb, int offset, int convert, (*items->hf_index == -1) ? "String Param" : proto_registrar_get_name(*items->hf_index), - string); + string ? string : "(null)"); items++; } else { offset = (*items->func)(tvb, offset, 0, |