1437796 - stylo-chrome: Conditional jump or move depends on uninitialised value(s)

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, enhancement)

Description

[Xidorn Quan [:xidorn] UTC+10]

There are several: https://treeherder.mozilla.org/logviewer.html#?job_id=161882929&repo=try&lineNumber=34334

We should probably just whitelist them like usual...

Comment 1

[Xidorn Quan [:xidorn] UTC+10]

Attachment: Bug 1437796 - Add two valgrind suppressions for stylo-chrome.

Review commit: https://reviewboard.mozilla.org/r/220052/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/220052/

Comment 2

[Xidorn Quan [:xidorn] UTC+10]

Comment on attachment 8950807 [details]
Bug 1437796 - Add two valgrind suppressions for stylo-chrome.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/220052/diff/1-2/

Comment 3

[Xidorn Quan [:xidorn] UTC+10]

Comment on attachment 8950807 [details]
Bug 1437796 - Add two valgrind suppressions for stylo-chrome.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/220052/diff/2-3/

Comment 4

[Xidorn Quan [:xidorn] UTC+10]

Julian, I updated the patch to include three lines for the second case as well. Could you have a look again?

Comment 5

[Julian Seward [:jseward]]

Comment on attachment 8950807 [details]
Bug 1437796 - Add two valgrind suppressions for stylo-chrome.

https://reviewboard.mozilla.org/r/220052/#review226498

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9e7df0a6dd52
Add two valgrind suppressions for stylo-chrome. r=jseward

Comment 7

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/9e7df0a6dd52