Home page logo
/

wireshark logo Wireshark mailing list archives

Commit r51222 triggered a coverity message:wq
From: Joerg Mayer <jmayer () loplof de>
Date: Tue, 13 Aug 2013 12:26:15 +0200

Hello,

commit

------------------------------------------------------------------------
r51222 | martinm | 2013-08-08 18:56:41 +0200 (Do, 08 Aug 2013) | 1 line

Apply same changes to common stats to tshark tap
------------------------------------------------------------------------

Seems to have triggered some coverity message. In case there really isn't
a break missing then maybe adding a fallthrough comment would be in order.

Ciao
    Jörg

----- Forwarded message from scan-admin () coverity com -----

From: scan-admin () coverity com
To: jmayer () loplof de
CC:
Subject: New Defects reported by Coverity Scan for Wireshark
Date: Thu, 08 Aug 2013 18:26:14 -0700


Hi,

Please find the latest report on new defect(s) introduced to Wireshark found with Coverity Scan

Defect(s) Reported-by: Coverity Scan


** CID 1061546: Missing break in switch (MISSING_BREAK)


CID 1061546: Missing break in switch (MISSING_BREAK)

/ui/cli/tap-macltestat.c: 246 ( unterminated_case)
   243                hs->common_stats.pch_bytes += si->single_number_of_bytes;
   244                hs->common_stats.pch_paging_ids += si->number_of_paging_ids;
   245                return 1;
This case (value 4) is not terminated by a 'break' statement.
   246            case SI_RNTI:
   247                hs->common_stats.sib_frames++;
   248                hs->common_stats.sib_bytes += si->single_number_of_bytes;
   249            case NO_RNTI:
   250                hs->common_stats.mib_frames++;


/ui/cli/tap-macltestat.c: 249 ( fallthrough)
   246            case SI_RNTI:
   247                hs->common_stats.sib_frames++;
   248                hs->common_stats.sib_bytes += si->single_number_of_bytes;
The above case falls through to this one.
   249            case NO_RNTI:
   250                hs->common_stats.mib_frames++;
   251                return 1;
   252            case RA_RNTI:
   253                hs->common_stats.rar_frames++;


--
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

  By Date           By Thread  

Current thread:
[ Nmap | Sec Tools | Mailing Lists | Site News | About/Contact | Advertising | Privacy ]