URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=dfa4125b70b61d4b58ba177860a0607724b39a8b
Submitter: "Pascal Quantin <pascal@xxxxxxxxxxxxx>"
Changed: branch: master-3.0
Repository: wireshark
Commits:
dfa4125 by Mark Vitale (mvitale@xxxxxxxxxxxxxx):
afs: correctly calulate padding for strings
length+value strings in AFS are padded to a multiple of 4 octets.
A bug caused the dissector to add an extra 4-octet pad when no padding
was required. This causes any field after the string to be
mis-dissected, resulting in a false alarm "Malformed packet".
Correct the padding logic to eliminate the false alarm.
Change-Id: I2edc58f20830c2df99d87cdd7d0cbf3bc9b92991
Reviewed-on: https://code.wireshark.org/review/36297
Reviewed-by: Tomasz Moń <desowin@xxxxxxxxx>
Petri-Dish: Tomasz Moń <desowin@xxxxxxxxx>
Tested-by: Petri Dish Buildbot
Reviewed-by: Pascal Quantin <pascal@xxxxxxxxxxxxx>
(cherry picked from commit 77d2bf81a75f682cf09f07ac02b2f4cb92324a59)
Reviewed-on: https://code.wireshark.org/review/36316
Actions performed:
from fa622cb gitlab-ci: remove unnecessary jobs, upgrade versions
add dfa4125 afs: correctly calulate padding for strings
Summary of changes:
epan/dissectors/packet-afs.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)