[Bug 4323] NetworkPkg/Library/DxeHttpLib: Coverity scan flags MISSING_BREAK issue


bugzilla-daemon@...
 

https://bugzilla.tianocore.org/show_bug.cgi?id=4323

--- Comment #1 from Veeresh Sangolli <veeresh.sangolli@...> ---
Created attachment 1313
--> https://bugzilla.tianocore.org/attachment.cgi?id=1313&action=edit
Proposed patch to resolve the Missing_Break issue pointed by Coverity for
NetworkPkg

The function HttpParseMessageBody has a switch-case on Parser->State
in which

case BodyParserChunkSizeStart: falls through to
case BodyParserChunkSize:

While this may be intentional, it is not evident to any general code
reader as well as any static analyzer tool. Just adding

// No break; here as this is an intentional fallthrough.

as comment in between makes any reader as well as Coverity happy.

--
You are receiving this mail because:
You are on the CC list for the bug.