diff options
| author | Jeff Morriss <jeff.morriss@ulticom.com> | 2010-05-17 21:49:09 +0000 |
|---|---|---|
| committer | Jeff Morriss <jeff.morriss@ulticom.com> | 2010-05-17 21:49:09 +0000 |
| commit | d1573b53c076e933d8bd6bcdaf3ca6fb04ccad82 (patch) | |
| tree | 01b3ba1b28efff6b5a125a129589148a38ceb053 /epan/dissectors/packet-smb-pipe.c | |
| parent | 734fb0725f40ed83cd283d41580b2b88dcd736e4 (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...)
svn path=/trunk/; revision=32848
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, |