minissdp: fix potential out-of-bounds read
albumart: remove logically dead code
cleanup: avoid dereferencing NULL pointers
cleanup: remove unnecessary sbeacon set
sql: use va_end() where appropriate
utils: add noalloc flag to modifyString()

This should make it more straightforward for code analyzers to
tell if the function is going to allocate memory.
albumart: use a separate variable for temporary pointer storage

Re-using the same one confuses some static code analyzers, and
is a bit less readable.
http: fix error handling in the fallback read/write path
tivo: clean up some error checking
sql: introduce sql_get_int64_field()

Returns int64_t instead of int.
subtitles: accept smi in addition to srt
Type data pointer variables as uint8_t.
minissdp: fix minor uninitialized access
log: replace a few fprintf's with DPRINTF's.
Merge commit '750917f7ea9876d7a97f48aa4b22097fcf7958d9'
Minor code simplification.
clients: add BubbleUPnP detection and enable subtitle support
scanner: check defined media types before parsing files
Fix a few memory leaks in some error conditions during image manipulation.
Allow the user to specify an arbitrary root container.