Sylvestre Ledru [:Sylvestre] Assignee
	Description • 7 years ago

with gcc 8-20180110-1
 In file included from /root/firefox-gcc-last/security/sandbox/linux/reporter/SandboxReporter.h:13,
                  from /root/firefox-gcc-last/security/sandbox/linux/reporter/SandboxReporter.cpp:7:
 /root/firefox-gcc-last/obj-x86_64-pc-linux-gnu/dist/include/mozilla/StaticPtr.h: In constructor 'mozilla::StaticAutoPtr<T>::StaticAutoPtr() [with T = mozilla::SandboxReporter]':
 /root/firefox-gcc-last/obj-x86_64-pc-linux-gnu/dist/include/mozilla/StaticPtr.h:48:17: error: '*<unknown>.mozilla::StaticAutoPtr<mozilla::SandboxReporter>::mRawPtr' is used uninitialized in this function [-Werror=uninitialized]
      MOZ_ASSERT(!mRawPtr);
                  ^~~~~~~
 cc1plus: all warnings being treated as errors

Not sure if this is a bug in the compiler false positive or not.

	James Cheng[:JamesCheng]
	Comment 1 • 7 years ago

I think we intended not to assign the value for

https://searchfox.org/mozilla-central/rev/41925c0b6c6d58578690121de439d2a8d3d690f3/xpcom/base/StaticPtr.h#48

we assume client should use this object as a global object that means it will be located in BSS section with auto zero initialization.

How could we work around for this warning?

	Nathan Froyd [:froydnj]
	Comment 2 • 7 years ago

Comment 1 is correct; mRawPtr will always be zero-initialized by the runtime system.  But the compiler can't know that we're always going to use these objects at file scope, so it complains.

#pragma GCC warning disable seems like a reasonable workaround.  Want to try that, Sylvestre?

	Sylvestre Ledru [:Sylvestre] Assignee
	Comment 4 • 7 years ago

Is that what you are expecting? Thanks!

	Nathan Froyd [:froydnj]
	Comment 5 • 7 years ago
mozreview-review

Comment on attachment 8942913 [details]
Bug 1430729 - Ignore an incorrect warning with gcc (-Wuninitialized)

https://reviewboard.mozilla.org/r/213182/#review218858

Thank you!

	Pulsebot
	Comment 6 • 7 years ago

Pushed by sledru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/55b7690ea8c5
Ignore an incorrect warning with gcc (-Wuninitialized) r=froydnj

	Andreea Pavel [:apavel]
	Comment 7 • 7 years ago

Backed out for build bustage z:\build\build\src\obj-firefox\dist\include\mozilla/StaticPtr.h(107): error C2220 on a CLOSED TREE

Push that cause the bustages: https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=55b7690ea8c554e12d11ea015c90a0e2bd115389

Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=156599549&repo=autoland&lineNumber=7094

Backout: https://hg.mozilla.org/integration/autoland/rev/b2588a5584c200b9dac8a6a0b0500f043cd335fe

	Pulsebot
	Comment 9 • 7 years ago

Pushed by sledru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/263b74db7a7c
Ignore an incorrect warning with gcc (-Wuninitialized) r=froydnj

	Sylvestre Ledru [:Sylvestre] Assignee
	Comment 10 • 7 years ago

According to the autoland CI, we are much better now!

	Cosmin Sabou [:CosminS]
	Comment 11 • 7 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/263b74db7a7c