Wireshark · Wireshark-bugs: [Wireshark-bugs] [Bug 6844] Universal Alcatel Protocol - Reloaded

[bugzilla-daemon@xxxxxxxxxxxxx]

https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6844

--- Comment #26 from Bill Meier <wmeier@xxxxxxxxxxx> 2012-06-01 08:56:10 PDT ---
Some questions based on working on packet-ua3g.c

1. There are a number of cases with code as follows:

   for (i = 1; i <= parameter_length; i++) {
       if (isprint(tvb_get_guint8(tvb, offset + 1 + i)))
           ep_strbuf_append(strbuf, tvb_get_const_stringz(tvb, offset+1+i,
NULL));
       else
           ep_strbuf_append(strbuf, tvb_get_const_stringz(tvb, offset+1+i,
NULL));
                            }

   a. The strings stored are the same for both branches
   b. The 'stringz' strings are generated starting at offsets 1,2,3,4,5,...

   Neither of the above seem right. If not, what is the correct code ?

2. In decode_lcd_line_cmd() the string stored in strbuf is never used
   and no value is stored in str_ascii before use.

   Is str_ascii supposed to be generated from the strbuf or something ?

3. In decode_dwl_special_char(), the strings stored in strbuf are never 
   actually used.

   /* The following loop will draw a picture of the character with "spaces" and
"o" */

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