ANNOUNCEMENT: Live Wireshark University & Allegro Packets online APAC Wireshark Training Session
April 17th, 2024 | 14:30-16:00 SGT (UTC+8) | Online

Wireshark-bugs: [Wireshark-bugs] [Bug 5630] new_packet_list: LUA dissector - gibberish in "Info"

Date: Wed, 2 Feb 2011 17:18:31 -0800 (PST)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5630

--- Comment #4 from yuri@xxxxxxxxxxxx 2011-02-02 17:18:30 PST ---
Using :set() does not work around the issue I've reported - the attached LUA
uses it already - pinfo.cols.info:set()

(In reply to comment #3)
> I ran into this issue today as well - I suspect that this is due to the use of
> col_set_str in wslua_pinfo.c:Columns__newindex
> 
> col_set_str simply stores the pointer passed to it, and luaL_checkstring
> returns a temporary pointer. Changing col_set_str to col_add_str in
> Columns_newindex seems to fix the issue, but I haven't tested it fully.
> 
> As a workaround - use pinfo.cols.src:set("THE STRING") rather than
> pinfo.cols.src="THE STRING". This method of assignment uses col_add_str which
> stores the string internally rather than by storing the pointer.

-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.