Seeing crashes in NFC Shutdown due to sNfcConsumer being NULL. This seems to occur if there are other problems in SystemWorkerManager initialization and it bails out and calls NfcConsumer::Shutdown in the exit path. Program received signal SIGSEGV, Segmentation fault. 0xb4936096 in mozilla::ipc::NfcConsumer::Shutdown () at ../../../src/ipc/nfc/Nfc.cpp:314 314 sNfcConsumer->mShutdown = true; (gdb) back #0 0xb4936096 in mozilla::ipc::NfcConsumer::Shutdown () at ../../../src/ipc/nfc/Nfc.cpp:314 #1 0xb4f7b35e in mozilla::dom::gonk::SystemWorkerManager::Shutdown (this=0x1669598) at ../../../../src/dom/system/gonk/SystemWorkerManager.cpp:132 #2 0xb4f7b6fa in mozilla::dom::gonk::SystemWorkerManager::FactoryCreate () at ../../../../src/dom/system/gonk/SystemWorkerManager.cpp:161 #3 0xb4d9eafc in SystemWorkerManagerConstructor (aResult=0xbe9109f8, aIID=..., aOuter=<optimized out>) at ../../../src/layout/build/nsLayoutModule.cpp:290 #4 SystemWorkerManagerConstructor (aOuter=<optimized out>, aIID=..., aResult=0xbe9109f8) at ../../../src/layout/build/nsLayoutModule.cpp:291

Comment on attachment 8381384 [details] [diff] [review] Guard against crash in NFC shutdown when the sNfcConsumer is NULL Review of attachment 8381384 [details] [diff] [review]: ----------------------------------------------------------------- Nice catch. Add r=me. May I know which device you're testing on? Nexus 4?

(In reply to Yoshi Huang[:allstars.chh] from comment #2) > Comment on attachment 8381384 [details] [diff] [review] > Guard against crash in NFC shutdown when the sNfcConsumer is NULL > > Review of attachment 8381384 [details] [diff] [review]: > ----------------------------------------------------------------- > > Nice catch. Add r=me. Thank you for the quick review. > May I know which device you're testing on? Nexus 4? Yes, the Nexus 4. btw This path was taken when Wifi debugging was enabled, see bug 976455.

Hi Douglas Due to the new policy we cannot land patches right now. Let's ask Ken for this.

After discussing with Yoshi, it causes regression.

After discussed with Ken, nominate this with 1.3+

It is 1.3+.

Comment on attachment 8381384 [details] [diff] [review] Guard against crash in NFC shutdown when the sNfcConsumer is NULL [Approval Request Comment] Bug caused by (feature/regressing bug #): Bug 933635 User impact if declined: Crash due to SIGSEGV Testing completed: verified on Nexus-4 Risk to taking this patch (and alternatives if risky): Low. This patch just checks if the pointer is valid or not before accessing it. String or UUID changes made by this patch: No