Blacklist nacl_message_scanner for UBSan.

Blacklist nacl_message_scanner for UBSan.

The code is doing unsafe (and potentially wrong, if the compiler comes to the play)
casts, which UBSan and CFI are not happy about. CFI already has this
file blacklisted, it's now the time for UBSan to join the party.

BUG=520760, 607996
Committed: https://crrev.com/7bacd498506b1089ec53cddf76152baa4840bc24
Cr-Commit-Position: refs/heads/master@{#429696}

[krasin1, 2016-11-03 20:55:47]

krasin@chromium.org changed reviewers:
+ dpranke@chromium.org

[Dirk Pranke, 2016-11-03 21:00:29]

lgtm, except I think you want your CL description to say "now the time", not
"not the time"?

[krasin1, 2016-11-03 21:01:55]

Description was changed from

==========
Blacklist nacl_message_scanner for UBSan.

The code is doing unsafe (and potentially wrong, if the compiler comes to the
play)
casts, which UBSan and CFI are not happy about. CFI already has this
file blacklisted, it's not the time for UBSan to join the party.

BUG=520760,607996
==========

to

==========
Blacklist nacl_message_scanner for UBSan.

The code is doing unsafe (and potentially wrong, if the compiler comes to the
play)
casts, which UBSan and CFI are not happy about. CFI already has this
file blacklisted, it's now the time for UBSan to join the party.

BUG=520760,607996
==========

[krasin1, 2016-11-03 21:02:04]

On 2016/11/03 21:00:29, Dirk Pranke wrote:
> lgtm, except I think you want your CL description to say "now the time", not
> "not the time"?

Thank you. Fixed.

[krasin1, 2016-11-03 21:02:09]

The CQ bit was checked by krasin@chromium.org

[commit-bot: I haz the power, 2016-11-03 21:03:30]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-03 21:13:19]

Description was changed from

==========
Blacklist nacl_message_scanner for UBSan.

The code is doing unsafe (and potentially wrong, if the compiler comes to the
play)
casts, which UBSan and CFI are not happy about. CFI already has this
file blacklisted, it's now the time for UBSan to join the party.

BUG=520760,607996
==========

to

==========
Blacklist nacl_message_scanner for UBSan.

The code is doing unsafe (and potentially wrong, if the compiler comes to the
play)
casts, which UBSan and CFI are not happy about. CFI already has this
file blacklisted, it's now the time for UBSan to join the party.

BUG=520760,607996
==========

[commit-bot: I haz the power, 2016-11-03 21:13:19]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-03 21:15:32]

Description was changed from

==========
Blacklist nacl_message_scanner for UBSan.

The code is doing unsafe (and potentially wrong, if the compiler comes to the
play)
casts, which UBSan and CFI are not happy about. CFI already has this
file blacklisted, it's now the time for UBSan to join the party.

BUG=520760,607996
==========

to

==========
Blacklist nacl_message_scanner for UBSan.

The code is doing unsafe (and potentially wrong, if the compiler comes to the
play)
casts, which UBSan and CFI are not happy about. CFI already has this
file blacklisted, it's now the time for UBSan to join the party.

BUG=520760,607996

Committed: https://crrev.com/7bacd498506b1089ec53cddf76152baa4840bc24
Cr-Commit-Position: refs/heads/master@{#429696}
==========

[commit-bot: I haz the power, 2016-11-03 21:15:33]

Patchset 1 (id:??) landed as
https://crrev.com/7bacd498506b1089ec53cddf76152baa4840bc24
Cr-Commit-Position: refs/heads/master@{#429696}