diff options
author | guy <guy@f5534014-38df-0310-8fa8-9805f1628bb7> | 2005-08-06 21:38:43 +0000 |
---|---|---|
committer | guy <guy@f5534014-38df-0310-8fa8-9805f1628bb7> | 2005-08-06 21:38:43 +0000 |
commit | 4c1ef0ec844a669848ca1aa2d65de21faf545e7e (patch) | |
tree | 5bcb5812006ce622179a9c732c50de8788f14c95 /epan/exceptions.h | |
parent | e281334054b428bf6beedf9092421bd06febbabf (diff) |
If a dissector writer directly throws DissectorError, don't punish the
user by crashing due to a null pointer dereference, punish the dissector
writer by saying he or she didn't bother providing an explanation of the
error.
The exception message isn't a const string any more; make the
"exception_message" argument to "show_exception()" a "char *", not a
"const char *".
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@15251 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'epan/exceptions.h')
-rw-r--r-- | epan/exceptions.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/epan/exceptions.h b/epan/exceptions.h index de223f4ebd..a3d814cba9 100644 --- a/epan/exceptions.h +++ b/epan/exceptions.h @@ -48,9 +48,9 @@ /** A bug was detected in a dissector. - DO NOT throw this with THROW(); the handler expects there to be a - message, and even if it didn't, the developers expect there to be - a message to make it easier to figure out what the problem is. + DO NOT throw this with THROW(); that means that no details about + the dissector error will be reported. (Instead, the message will + blame you for not providing details.) Instead, use the DISSECTOR_ASSERT(), etc. macros in epan/proto.h. **/ |