Dont try to use class members after the class has been deleted.

2003-06-13  Marco Pesenti Gritti  <marco@it.gnome.org>

	* embed/mozilla/ProgressListener.cpp:

	Dont try to use class members after the class
	has been deleted.

	* embed/mozilla/mozilla-embed-persist.cpp:

	Fix use of uninitialized var.

3 files changed, 15 insertions, 2 deletions

diff --git a/ChangeLog b/ChangeLog
index d26a4fc95..53f1e9691 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,14 @@
+2003-06-13  Marco Pesenti Gritti  <marco@it.gnome.org>
+
+	* embed/mozilla/ProgressListener.cpp:
+
+	Dont try to use class members after the class
+	has been deleted.
+
+	* embed/mozilla/mozilla-embed-persist.cpp:
+
+	Fix use of uninitialized var.
+
 2003-06-12  Marco Pesenti Gritti  <marco@it.gnome.org>
 
 	* embed/mozilla/mozilla-embed.cpp:
diff --git a/embed/mozilla/ProgressListener.cpp b/embed/mozilla/ProgressListener.cpp
index 20ad8e88c..a8f949dce 100644
--- a/embed/mozilla/ProgressListener.cpp
+++ b/embed/mozilla/ProgressListener.cpp
@@ -353,7 +353,9 @@ NS_IMETHODIMP GProgressListener::OnStateChange (nsIWebProgress *aWebProgress,
 			}
 		
 			mozilla_embed_persist_completed
-				(MOZILLA_EMBED_PERSIST (mEphyPersist));	
+				(MOZILLA_EMBED_PERSIST (mEphyPersist));
+
+			return NS_OK;	
 		}
 		
 		if (!mNoDialog)
diff --git a/embed/mozilla/mozilla-embed-persist.cpp b/embed/mozilla/mozilla-embed-persist.cpp
index 4677613ea..8a42e5b03 100644
--- a/embed/mozilla/mozilla-embed-persist.cpp
+++ b/embed/mozilla/mozilla-embed-persist.cpp
@@ -183,7 +183,7 @@ impl_save (EphyEmbedPersist *persist)
 	}
 
         nsCOMPtr<nsILocalFile> file;
-       	NS_NewLocalFile(NS_ConvertUTF8toUCS2(filename), PR_TRUE, getter_AddRefs(file)); 
+       	rv = NS_NewLocalFile(NS_ConvertUTF8toUCS2(filename), PR_TRUE, getter_AddRefs(file)); 
         if (NS_FAILED(rv) || !file) return G_FAILED;
 
 	nsCOMPtr<nsILocalFile> path;