Opened 11 years ago
Closed 10 years ago
Last modified 10 years ago
#11610 closed Patch - Bug Fix (fixed)
Patch to check fcntl success in mythsystemunix.cpp - fixes coverity 1028691
Reported by: | Owned by: | Karl Egly | |
---|---|---|---|
Priority: | minor | Milestone: | 0.28 |
Component: | MythTV - MythSystem | Version: | Master Head |
Severity: | medium | Keywords: | |
Cc: | Ticket locked: | no |
Description
Static analysis (coverity) detected that the system call fcntl in mythsystemunix.cpp did not check its return value. This patch adds in that check.
This patch also implements the more robust setting of the fd flags (get the current flags, or'ing in the NONBLOCK value, and then setting the flags). This is the "correct" way to set flags per the docs (even though in this case it is probably overkill).
Fixes coverity 1028691
github patch: https://github.com/garybuhrmaster/mythtv/commit/26472f379970d6be81c77e8d2f311dccfd25ad47
github git-am patch: https://github.com/garybuhrmaster/mythtv/commit/26472f379970d6be81c77e8d2f311dccfd25ad47.patch
In 246c40737794963306bacfb6960008ad23eb7ebc/mythtv: