diff --git a/storage/test/storage_test_harness_tail.h b/storage/test/storage_test_harness_tail.h
index b3a532a28375..65a7be27ad76 100644
--- a/storage/test/storage_test_harness_tail.h
+++ b/storage/test/storage_test_harness_tail.h
@@ -49,7 +49,6 @@ int
 main(int aArgc,
      char **aArgv)
 {
-  return 0;
   ScopedXPCOM xpcom(TEST_NAME);
 
   for (size_t i = 0; i < NS_ARRAY_LENGTH(gTests); i++)
diff --git a/toolkit/components/places/src/History.cpp b/toolkit/components/places/src/History.cpp
index ecc6a5941da6..dcc51330c789 100644
--- a/toolkit/components/places/src/History.cpp
+++ b/toolkit/components/places/src/History.cpp
@@ -79,7 +79,6 @@ public:
   static nsresult Start(nsIURI* aURI)
   {
     NS_PRECONDITION(aURI, "Null URI");
-    return NS_OK;
 
 #ifdef MOZ_IPC
   // If we are a content process, always remote the request to the
diff --git a/toolkit/components/places/tests/cpp/places_test_harness_tail.h b/toolkit/components/places/tests/cpp/places_test_harness_tail.h
index 9975253303a5..68c9aca64342 100644
--- a/toolkit/components/places/tests/cpp/places_test_harness_tail.h
+++ b/toolkit/components/places/tests/cpp/places_test_harness_tail.h
@@ -96,7 +96,6 @@ int
 main(int aArgc,
      char** aArgv)
 {
-  return 0;
   ScopedXPCOM xpcom(TEST_NAME);
 
   do_test_pending();