typo in card id check and proper dereference in null pointer check

[Erik Hovland <erik@…>]

In libmythtv in the member function CardInput::sourceFetch() in the file videosource.cpp there is a check of the pointer cardid at line 2258. But the pointer cardid has already been dereferenced, therefore the check doesn't really mean anything. Instead what the function is likely to need to do is check if the crdid is 0 (invalid) or not.

In libmythtv in the function sub_readtext in the file xine_demux_sputext.c there is a check of the double pointer dest. This is directly after a call to malloc for the pointer *dest. It is likely that the caller wanted to check the validity of the *dest and not dest. Since that is the pointer that might be null.

[Erik Hovland <erik@…>]

fixes typo in one 'if' and a wrong dereference in another 'if'

[danielk]

(In [16197]) Fixes #4737. Fixes a null pointer check so it actually works, and fixes a typo that lead to an always true null pointer check instead of a null value check. Patch from Erik Hovland.