From Emil Wojak via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5391 :

I'd like to share my enhancements to the TDS dissector with everyone.
The list of improvements follows:
- nearly complete dissection of RPC calls,
- detection and dissection of the ALL_HEADERS rule,
- corrected some existing proto_tree fields to support filters,
- other minor fixes where the interpretation of data conflicted with the
  official documentation from MS.

I tested the new code on a variety of different TDS captures with many diverse
RPC calls. The code compiles and works on 32-bit Linux, I didn't check those
changes on other platforms though.

From me:

- terminate all value_strings
- change ++*offset to *offset += 1 (I think that's more readable)
- replace all the dissector assertions which could be caused by malformed
  packets with expert infos
- Don't throw ReportedBoundsError when the packets have unexpected data in
  them, just report an expert info and continue on


git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@35007 f5534014-38df-0310-8fa8-9805f1628bb7

1 files changed, 4 insertions, 0 deletions

diff --git a/AUTHORS b/AUTHORS
index 1f973338a9..179c341ce2 100644
--- a/AUTHORS
+++ b/AUTHORS
@@ -3202,6 +3202,10 @@ Stéphane Bryant <stephane [AT] glycon.org> {
 	RELOAD Framing dissector
 }
 
+Emil Wojak <emil [AT] wojak.eu>
+	TDS dissector improvements
+}
+
 and by:
 
 Pavel Roskin            <proski [AT] gnu.org>