Project

General

Profile

Actions

Bug #1670

closed

Modbus compiler warnings on Fedora 23

Added by Peter Manev almost 9 years ago. Updated almost 9 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
Affected Versions:
Effort:
Difficulty:
Label:

Description

When compiling 3.0RC3 from source on Fedora 23 -

[root@localhost suricata-3.0RC3]# uname -a
Linux localhost.localdomain 4.2.7-300.fc23.x86_64 #1 SMP Wed Dec 9 22:28:30 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux

the following warnings are present about modbus:

  CC       app-layer-modbus.o
app-layer-modbus.c: In function ‘ModbusParseWriteRequest’:
app-layer-modbus.c:736:31: warning: ‘byte’ may be used uninitialized in this function [-Wmaybe-uninitialized]
                 tx->data[i] = (uint16_t) byte;
                               ^
app-layer-modbus.c: In function ‘ModbusParseResponse’:
app-layer-modbus.c:614:24: warning: ‘count’ may be used uninitialized in this function [-Wmaybe-uninitialized]
                 (count != (2 * (tx->read.quantity))))
                        ^
app-layer-modbus.c:599:13: note: ‘count’ was declared here
     uint8_t count;
             ^
app-layer-modbus.c:1164:24: warning: ‘mei’ may be used uninitialized in this function [-Wmaybe-uninitialized]
                     if (mei != tx->mei)
                        ^
app-layer-modbus.c:1121:49: note: ‘mei’ was declared here
     uint8_t     count, error = FALSE, function, mei;
                                                 ^
app-layer-modbus.c:1156:21: warning: ‘count’ may be used uninitialized in this function [-Wmaybe-uninitialized]
                     ModbusCheckHeaderLength(modbus, tx->length, 2 + count);
                     ^
app-layer-modbus.c:1121:17: note: ‘count’ was declared here
     uint8_t     count, error = FALSE, function, mei;
                 ^
app-layer-modbus.c:499:5: warning: ‘exception’ may be used uninitialized in this function [-Wmaybe-uninitialized]
     switch (exception) {
     ^
app-layer-modbus.c:493:13: note: ‘exception’ was declared here
     uint8_t exception;
             ^
app-layer-modbus.c:1138:13: warning: ‘function’ may be used uninitialized in this function [-Wmaybe-uninitialized]
             switch(function) {
             ^
app-layer-modbus.c:1121:39: note: ‘function’ was declared here
     uint8_t     count, error = FALSE, function, mei;
                                       ^
app-layer-modbus.c: In function ‘ModbusParseRequest’:
app-layer-modbus.c:1040:13: warning: ‘count’ may be used uninitialized in this function [-Wmaybe-uninitialized]
             ModbusCheckHeaderLength(modbus, tx->length, 2 + count);
             ^
app-layer-modbus.c:964:17: note: ‘count’ was declared here
     uint8_t     count;
                 ^
Actions

Also available in: Atom PDF