1376643 - Static analysis fixes in security/sandbox/

Status: RESOLVED FIXED

(Core :: Security: Process Sandboxing, enhancement)

Description

[Jan Keromnes [:janx]]

There are several opportunities for small automated static analysis improvements in security/sandbox/.

Comment 1

[Jan Keromnes [:janx]]

Attachment: Use 'nullptr' where applicable in security/sandbox/.

Gian-Carlo, please have a look.

Comment 2

[Jan Keromnes [:janx]]

Attachment: Use 'override' and '= default' on applicable methods in security/sandbox/.

Comment 3

[Jan Keromnes [:janx]]

Thanks!

https://treeherder.mozilla.org/#/jobs?repo=try&revision=a3f27f645ef3

Comment 4

[Jan Keromnes [:janx]]

Try is green apart from some unrelated-looking oranges.

Sheriffs, please land these patches.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/de8eb156e157
Use 'nullptr' where applicable in security/sandbox/. r=gcp
https://hg.mozilla.org/integration/mozilla-inbound/rev/f551e82578c1
Use 'override' and '= default' on applicable methods in security/sandbox/. r=gcp

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/de8eb156e157
https://hg.mozilla.org/mozilla-central/rev/f551e82578c1