[mythtv-commits] Ticket #11613: Patch to initialize (and properly copy) the aes key/iv values in httplivestreambuffer.cpp - fixes coverity 1023902
MythTV
noreply at mythtv.org
Wed Jun 19 21:17:55 UTC 2013
#11613: Patch to initialize (and properly copy) the aes key/iv values in
httplivestreambuffer.cpp - fixes coverity 1023902
-------------------------------------+-------------------------------------
Reporter: Gary Buhrmaster | Type: Patch - Bug Fix
<gary.buhrmaster@…> | Priority: minor
Status: new | Component: MythTV - HTTP
Milestone: unknown | Streaming
Version: Master Head | Severity: medium
Keywords: | Ticket locked: 0
-------------------------------------+-------------------------------------
Static analysis indicated that the key/iv variables were not initialized.
Strictly speaking, they do not have to be, since they are protected
by a boolean flag variable. But to make coverity happy, initialize them
(note that the init value of zero is not any more correct or wrong than a
random uninitialized value for the cryptographic operations).
In addition, the assignment operator did not copy the iv (even though it
copied the flag variable). This looks like a real bug.
Fixes coverty 1023902
github ref:
https://github.com/garybuhrmaster/mythtv/commit/c084f6d6a2e2338744b86baa03bce3816caea9db
githum git-am ref:
https://github.com/garybuhrmaster/mythtv/commit/c084f6d6a2e2338744b86baa03bce3816caea9db.patch
--
Ticket URL: <http://code.mythtv.org/trac/ticket/11613>
MythTV <http://www.mythtv.org>
MythTV Media Center
More information about the mythtv-commits
mailing list