add null checks
This commit is contained in:
parent
94451de6d2
commit
f8c88aa632
|
@ -95,6 +95,11 @@ namespace MediaBrowser.MediaEncoding.Encoder
|
||||||
int defaultImageExtractionTimeoutMs,
|
int defaultImageExtractionTimeoutMs,
|
||||||
bool enableEncoderFontFile, IEnvironmentInfo environmentInfo)
|
bool enableEncoderFontFile, IEnvironmentInfo environmentInfo)
|
||||||
{
|
{
|
||||||
|
if (jsonSerializer == null)
|
||||||
|
{
|
||||||
|
throw new ArgumentNullException("jsonSerializer");
|
||||||
|
}
|
||||||
|
|
||||||
_logger = logger;
|
_logger = logger;
|
||||||
_jsonSerializer = jsonSerializer;
|
_jsonSerializer = jsonSerializer;
|
||||||
ConfigurationManager = configurationManager;
|
ConfigurationManager = configurationManager;
|
||||||
|
@ -632,7 +637,7 @@ namespace MediaBrowser.MediaEncoding.Encoder
|
||||||
|
|
||||||
var result = _jsonSerializer.DeserializeFromStream<InternalMediaInfoResult>(process.StandardOutput.BaseStream);
|
var result = _jsonSerializer.DeserializeFromStream<InternalMediaInfoResult>(process.StandardOutput.BaseStream);
|
||||||
|
|
||||||
if (result.streams == null && result.format == null)
|
if (result == null || (result.streams == null && result.format == null))
|
||||||
{
|
{
|
||||||
throw new Exception("ffprobe failed - streams and format are both null.");
|
throw new Exception("ffprobe failed - streams and format are both null.");
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue
Block a user