Wireshark · Wireshark-commits: [Wireshark-commits] master 3868252: Lua: allow creating TVBs after calling other (Lua) dissectors.

[Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>]

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=38682523f9ce3ed378c782a848ba1ba5a6fe430f
Submitter: Peter Wu (peter@xxxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

3868252 by Jeff Morriss (jeff.morriss.ws@xxxxxxxxx):

    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.ws@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Peter Wu <peter@xxxxxxxxxxxxx>
    Tested-by: Peter Wu <peter@xxxxxxxxxxxxx>
    

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(-)