From Peter Harris via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4701 :
authormorriss <morriss@f5534014-38df-0310-8fa8-9805f1628bb7>
Wed, 21 Apr 2010 00:47:55 +0000 (00:47 +0000)
committermorriss <morriss@f5534014-38df-0310-8fa8-9805f1628bb7>
Wed, 21 Apr 2010 00:47:55 +0000 (00:47 +0000)
commit28aa4c6747457277f38499d90ed1dfda1cf63371
treefea4c614aa40104bf9f2d81a9953a1f22cc067b2
parent8f7662ab5018981b65218badbb10cb694ac2de50
From Peter Harris via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4701 :

The ChangeProperty request always calls LISTofBYTE, which always tries to
dissect at least one byte, even when the request is empty. This causes
Wireshark to put a big, red, scary "Malformed Packet" entry in the dissection.

Also, ChangeProperty only dissects (data_length) bytes. data_length is "number
of units", and units may be 16 or 32-bit entities. In this case, the dissected
data will be truncated.

First reported in https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4481#c10

The attached patch fixes these two bugs.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@32526 f5534014-38df-0310-8fa8-9805f1628bb7
epan/dissectors/packet-x11.c
epan/dissectors/x11-declarations.h
epan/dissectors/x11-fields
epan/dissectors/x11-register-info.h