[mythtv-firehose] mythtv branch master updated by bmeek. v0.28-pre-3734-g5ca0dde

Git Repo Owner noreply at mythtv.org
Wed Apr 6 23:33:42 UTC 2016

The branch, master has been updated on the
mythtv repository by gitolite user bmeek.
       via  5ca0ddeab014daaaa2d7e8eeb5bbf6b095202a86 (commit)
      from  05383188d1c786d98ee236c8d9e57e46c7c1283d (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 5ca0ddeab014daaaa2d7e8eeb5bbf6b095202a86
Author:    Bill Meek <bmeek at mythtv.org> at Wed, 6 Apr 2016 18:31:59 -0500
Committer: Bill Meek <bmeek at mythtv.org> at Wed, 6 Apr 2016 18:31:59 -0500
URL:       http://code.mythtv.org/cgit/mythtv/commit/?id=5ca0ddeab014daaaa2d7e8eeb5bbf6b095202a86

Services API: Video::UpdateVideoWatchedStatus can create empty DB rows
loadOneFromDatabase()'s result is checked to verify that the Video's
Id is valid. However, invalid Ids pass too. Then UpdateDatabase()
is called and an empty record is inserted into videometadata. Using
loadAllFromDatabase(), which is called loadOneFromDatabase() too,
allows a valid check of !metadata and now returns a 501 error.


Summary of changes:
 mythtv/programs/mythbackend/services/video.cpp |   20 ++++++++++----------
 1 files changed, 10 insertions(+), 10 deletions(-)


More information about the mythtv-firehose mailing list