Wireshark-bugs: [Wireshark-bugs] [Bug 6766] New: lua proto registration fails for uppercase prot

Date: Wed, 25 Jan 2012 08:27:30 -0800 (PST)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6766

           Summary: lua proto registration fails for uppercase proto /
                    g_ascii_strdown problem
           Product: Wireshark
           Version: SVN
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Major
          Priority: Low
         Component: Wireshark
        AssignedTo: bugzilla-admin@xxxxxxxxxxxxx
        ReportedBy: sbin23435@xxxxxxxxxxxxxx


Build Information:
TShark 1.7.1 (SVN Rev 40717 from /trunk)

Copyright 1998-2012 Gerald Combs <gerald@xxxxxxxxxxxxx> and contributors.
This is free software; see the source for copying conditions. There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Compiled (64-bit) with GLib 2.30.0, with libpcap (version unknown), with libz
1.2.3.4, with POSIX capabilities (Linux), with SMI 0.4.8, with c-ares 1.7.4,
with Lua 5.1, without Python, with GnuTLS 2.10.5, with Gcrypt 1.5.0, with MIT
Kerberos, with GeoIP.

Running on Linux 3.0.0-14-generic, with locale de_DE.UTF-8, with libpcap
version
1.1.1, with libz 1.2.3.4.

Built using gcc 4.6.1.

--
Changelog has an entry: s/g_strdown/g_ascii_strdown

this is WRONG. g_ascii_strdown doesn't inplace-lowercase but returns a new
string instead. if you have a call like

   Proto("MYPROTO", "description")

wireshark will refuse to register the proto because it contains upper case.

the code in wslua_proto.c is supposed to turn it into lowercase, but the
replacement of deprecated functions has gone awfully wrong there. might be
sensible to check for further code with such problems, e.g. uat.h looks like it
might be affected...

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