diff options
author | Guy Harris <guy@alum.mit.edu> | 2005-08-06 21:38:43 +0000 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2005-08-06 21:38:43 +0000 |
commit | 84cf5472e9fffefbd3e9984adb1a7ec9c4ebe33f (patch) | |
tree | 5bcb5812006ce622179a9c732c50de8788f14c95 /epan/exceptions.h | |
parent | f618b54d368be8ee5b26c982f492d4132cda1f16 (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 *".
svn path=/trunk/; revision=15251
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. **/ |