WOFF2: avoid a Msan error use-of-uninitialized-value in woff2_dec.cc

WOFF2: avoid a Msan error use-of-uninitialized-value in woff2_dec.cc

Uninitialized buffer is read to obtain x_min value when n_contours
is zero, but read value isn't used for the case. So, this isn't
real issue at all, but to make the Msan bot feeling happy, fix
the logic not to read if n_contours is zero.

BUG=602975
Committed: https://crrev.com/988efed96ca86663c64ad8d93e779a3a1c2f1be2
Cr-Commit-Position: refs/heads/master@{#388459}

[Takashi Toyoshima, 2016-04-18 13:21:48]

Description was changed from

==========
WOFF2: avoid a Msan error use-of-uninitialized-value in woff2_dec.cc

Uninitialized buffer is read to obtain x_min value when n_contours
is zero, but read value isn't used for the case. So, this isn't
real issue at all, but to make the Msan bot feeling happy, fix
the logic not to read if n_contours is zero.

BUG=602975
==========

to

==========
WOFF2: avoid a Msan error use-of-uninitialized-value in woff2_dec.cc

Uninitialized buffer is read to obtain x_min value when n_contours
is zero, but read value isn't used for the case. So, this isn't
real issue at all, but to make the Msan bot feeling happy, fix
the logic not to read if n_contours is zero.

BUG=602975
==========

[Takashi Toyoshima, 2016-04-18 13:21:48]

toyoshim@chromium.org changed reviewers:
+ ksakamoto@chromium.org

[Takashi Toyoshima, 2016-04-19 08:43:02]

let me publish this review request now, but probably you would like to take a
look after upstream review is finished.

[Takashi Toyoshima, 2016-04-20 03:54:07]

FYI, upstream review was finished with changes that are equivalent to the Patch
Set 2.

[Kunihiko Sakamoto, 2016-04-20 03:55:15]

lgtm

[Takashi Toyoshima, 2016-04-20 08:15:45]

The CQ bit was checked by toyoshim@chromium.org

[commit-bot: I haz the power, 2016-04-20 08:15:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1895043002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1895043002/20001

[commit-bot: I haz the power, 2016-04-20 10:11:09]

Description was changed from

==========
WOFF2: avoid a Msan error use-of-uninitialized-value in woff2_dec.cc

Uninitialized buffer is read to obtain x_min value when n_contours
is zero, but read value isn't used for the case. So, this isn't
real issue at all, but to make the Msan bot feeling happy, fix
the logic not to read if n_contours is zero.

BUG=602975
==========

to

==========
WOFF2: avoid a Msan error use-of-uninitialized-value in woff2_dec.cc

Uninitialized buffer is read to obtain x_min value when n_contours
is zero, but read value isn't used for the case. So, this isn't
real issue at all, but to make the Msan bot feeling happy, fix
the logic not to read if n_contours is zero.

BUG=602975
==========

[commit-bot: I haz the power, 2016-04-20 10:11:10]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-22 19:22:07]

Description was changed from

==========
WOFF2: avoid a Msan error use-of-uninitialized-value in woff2_dec.cc

Uninitialized buffer is read to obtain x_min value when n_contours
is zero, but read value isn't used for the case. So, this isn't
real issue at all, but to make the Msan bot feeling happy, fix
the logic not to read if n_contours is zero.

BUG=602975
==========

to

==========
WOFF2: avoid a Msan error use-of-uninitialized-value in woff2_dec.cc

Uninitialized buffer is read to obtain x_min value when n_contours
is zero, but read value isn't used for the case. So, this isn't
real issue at all, but to make the Msan bot feeling happy, fix
the logic not to read if n_contours is zero.

BUG=602975

Committed: https://crrev.com/988efed96ca86663c64ad8d93e779a3a1c2f1be2
Cr-Commit-Position: refs/heads/master@{#388459}
==========

[commit-bot: I haz the power, 2016-04-22 19:22:08]

Patchset 2 (id:??) landed as
https://crrev.com/988efed96ca86663c64ad8d93e779a3a1c2f1be2
Cr-Commit-Position: refs/heads/master@{#388459}