Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#11598 closed Patch - Bug Fix (fixed)

Patch to check the success of setsockopt in httprequest.cpp - fixes coverity 746751 and 746752

Reported by: Gary Buhrmaster <gary.buhrmaster@…> Owned by: dblain
Priority: minor Milestone: 0.27
Component: MythTV - UPnP Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Static analysis (coverity) reported a defect for not checking the return from setsockopt. Put in the check, and log if the setsockopt fails (unlikely).

Fixes coverity 746751 and 746752

github patch link:

https://github.com/garybuhrmaster/mythtv/commit/8ec9a2d38c3ce94249382dfdc9d8a6ab66831356

or, for the git-am format:

https://github.com/garybuhrmaster/mythtv/commit/8ec9a2d38c3ce94249382dfdc9d8a6ab66831356.patch

Change History (2)

comment:1 Changed 5 years ago by Gary Buhrmaster <gary.buhrmaster@…>

Resolution: fixed
Status: newclosed

In 6c84654b168a875ffdf9230e291ef630f21267af/mythtv:

Check the success of setsockopt

Static analysis (coverity) reported a defect for not checking
the return from setsockopt. Put in the check, and log if the
setsockopt fails (unlikely).

Fixes coverity 746751 and 746752
(cherry picked from commit 8ec9a2d38c3ce94249382dfdc9d8a6ab66831356)

Fixes #11598

Signed-off-by: Stuart Morgan <smorgan@…>

comment:2 Changed 5 years ago by paulh

Milestone: unknown0.27
Note: See TracTickets for help on using tickets.