Hi Richard, Le mar. 16 oct. 2018 à 17:26, Richard Sharpe <realrichardsha...@gmail.com> a écrit :
> Hi folks, > > I have always put a {0, NULL} element on the end of value strings, but > it seems the lua code for constructing value strings when you use > ProtoField(..., some_value_string) does not terminate the list with an > empty element, and my test did not cause a crash. > > Rather, I got an "unknown" field as expected. > > Can anyone confirm they are not needed? > With C code, their presence is mandatory to properly end the loop in try_val_to_str_idx/try_val64_to_str_idx functions without doing an out of bounds access. I have no checked the Lua wrapper code. BR, Pascal.
___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev@wireshark.org> Archives: https://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe