URL:
https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=38682523f9ce3ed378c782a848ba1ba5a6fe430f
Submitter: Peter Wu (pe...@lekensteyn.nl)
Changed: branch: master
Repository: wireshark
Commits:
3868252 by Jeff Morriss (jeff.morriss...@gmail.com):
Lua: allow creating TVBs after calling other (Lua) dissectors.
Don't set `lua_tvb` (or any of the other global variables) to NULL after a
Lua dissector is called: it's possible that the caller is also a Lua
dissector
which may want/need that (global) variable to still be set (to the value it
had before the sub-dissector was called).
This fixes the problem reported in:
https://ask.wireshark.org/questions/56110/lua-error-tvbs-can-only-be-created-and-used-in-dissectors
Making these variables not be a globals (as suggested at the top of
init_wslua.c) might be a better solution--for another day.
Change-Id: I14fb8ec35b62abeda3f3471a323b88c80537a06e
Reviewed-on: https://code.wireshark.org/review/18095
Petri-Dish: Jeff Morriss <jeff.morriss...@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-re...@wireshark.org>
Reviewed-by: Peter Wu <pe...@lekensteyn.nl>
Tested-by: Peter Wu <pe...@lekensteyn.nl>
Actions performed:
from 3a08906 Retrieve CLNP option values from the right place
adds 3868252 Lua: allow creating TVBs after calling other (Lua)
dissectors.
Summary of changes:
epan/wslua/init_wslua.c | 18 ++++++++++++------
1 file changed, 12 insertions(+), 6 deletions(-)
___________________________________________________________________________
Sent via: Wireshark-commits mailing list <wireshark-commits@wireshark.org>
Archives: https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-commits
mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe
