VP8DecoderImpl: Fix uninitialized memory crash

VP8DecoderImpl: Fix uninitialized memory crash

It is not safe to call vpx_codec_destroy if vpx_codec_dec_init failed,
because the |decoder_| memory will be uninitialized. See the bug for
more info.

BUG=chromium:663293
TBR=mflodman,tommi

Committed: https://chromium.googlesource.com/external/webrtc/+/ac061a9ab6ca6690acb6284c1ee34860927acfd7

[magjed_webrtc, 2016-12-13 10:52:34]

Description was changed from

==========
VP8DecoderImpl: Fix uninitialized memory crash

It is not safe to call vpx_codec_destroy if vpx_codec_dec_init failed,
because the |decoder_| memory will be uninitialized. See the bug for
more info.

BUG=chromium:663293
==========

to

==========
VP8DecoderImpl: Fix uninitialized memory crash

It is not safe to call vpx_codec_destroy if vpx_codec_dec_init failed,
because the |decoder_| memory will be uninitialized. See the bug for
more info.

BUG=chromium:663293
==========

[magjed_webrtc, 2016-12-13 10:52:34]

magjed@webrtc.org changed reviewers:
+ mflodman@webrtc.org, tommi@webrtc.org

[magjed_webrtc, 2016-12-13 10:52:45]

Description was changed from

==========
VP8DecoderImpl: Fix uninitialized memory crash

It is not safe to call vpx_codec_destroy if vpx_codec_dec_init failed,
because the |decoder_| memory will be uninitialized. See the bug for
more info.

BUG=chromium:663293
==========

to

==========
VP8DecoderImpl: Fix uninitialized memory crash

It is not safe to call vpx_codec_destroy if vpx_codec_dec_init failed,
because the |decoder_| memory will be uninitialized. See the bug for
more info.

BUG=chromium:663293
TBR=mflodman,tommi
==========

[magjed_webrtc, 2016-12-13 10:53:42]

Description was changed from

==========
VP8DecoderImpl: Fix uninitialized memory crash

It is not safe to call vpx_codec_destroy if vpx_codec_dec_init failed,
because the |decoder_| memory will be uninitialized. See the bug for
more info.

BUG=chromium:663293
TBR=mflodman,tommi
==========

to

==========
VP8DecoderImpl: Fix uninitialized memory crash

It is not safe to call vpx_codec_destroy if vpx_codec_dec_init failed,
because the |decoder_| memory will be uninitialized. See the bug for
more info.

BUG=chromium:663293
TBR=mflodman,tommi

Committed:
https://chromium.googlesource.com/external/webrtc/+/ac061a9ab6ca6690acb6284c1...
==========

[magjed_webrtc, 2016-12-13 10:53:43]

Committed patchset #1 (id:1) manually as
ac061a9ab6ca6690acb6284c1ee34860927acfd7 (presubmit successful).

[tommi, 2016-12-13 11:04:39]

lgtm

[mflodman, 2016-12-14 08:12:45]

lgtm