Re: [Wireshark-bugs] [Bug 7976] Lua code crashes wireshark after update to 1.8.3

[Joerg Mayer <jmayer () loplof de>]

On Tue, Dec 04, 2012 at 10:39:16PM +0000, bugzilla-daemon () wireshark org wrote:
> https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7976
>
> Jeff Morriss <jeff.morriss.ws () gmail com> changed:
>
>            What    |Removed                     |Added
> ----------------------------------------------------------------------------
>              Status|INCOMPLETE                  |RESOLVED
>          Resolution|---                         |FIXED
>
> --- Comment #17 from Jeff Morriss <jeff.morriss.ws () gmail com> ---
> The original fix did not check the condition in all cases.  r46302 (trunk) and
> r46303 (trunk-1.8).  So it should be fixed in 1.8.5.  If you want to try it out
> before then you can get an automated build from:
>
> http://www.wireshark.org/download/automated/
>
> I'm guessing this will be the complete fix, so closing again.

Just as an idea for an alternate solution: Create a constant string at the
top of the file, saying something like
"It is an error to leave the abbreviation field empty" and thenx, whenever
we encounter this error instead of calling WSLUA_ARG_ERROR or luaL_argerror
just do a g_strdup of that string into abbr?

Ciao
      Jörg
-- 
Joerg Mayer                                           <jmayer () loplof de>
We are stuck with technology when what we really want is just stuff that
works. Some say that should read Microsoft instead of technology.
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request () wireshark org?subject=unsubscribe