From 84ac9ca4601774906c034d41ef9ef3b1504ad199 Mon Sep 17 00:00:00 2001
From: "peterl%netscape.com"
 <peterl%netscape.com@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Sun, 27 Jun 1999 21:16:16 +0000
Subject: [PATCH] cleanup url listing

git-svn-id: svn://10.0.0.236/trunk@37143 18797224-902f-48f8-a5cc-f745e15eee43
---
 .../html/style/src/nsCSSStyleSheet.cpp        | 33 +++++++++++--------
 .../layout/html/style/src/nsCSSStyleSheet.cpp | 33 +++++++++++--------
 mozilla/layout/style/nsCSSStyleSheet.cpp      | 33 +++++++++++--------
 3 files changed, 60 insertions(+), 39 deletions(-)

diff --git a/mozilla/content/html/style/src/nsCSSStyleSheet.cpp b/mozilla/content/html/style/src/nsCSSStyleSheet.cpp
index 16d8b66ac48..7adddb8cf4e 100644
--- a/mozilla/content/html/style/src/nsCSSStyleSheet.cpp
+++ b/mozilla/content/html/style/src/nsCSSStyleSheet.cpp
@@ -2370,16 +2370,20 @@ void CSSStyleSheetImpl::List(FILE* out, PRInt32 aIndent) const
 
   fputs("CSS Style Sheet: ", out);
 #ifdef NECKO
-  char* buffer;
-  nsresult rv = mInner->mURL->GetSpec(&buffer);
-  fputs(buffer, out);
-  nsCRT::free(buffer);
+  char* urlSpec = nsnull;
+  nsresult rv = mInner->mURL->GetSpec(&urlSpec);
+  if (urlSpec) {
+    fputs(urlSpec, out);
+    nsCRT::free(urlSpec);
+  }
 #else
-  PRUnichar* buffer;
-  mInner->mURL->ToString(&buffer);
-  nsAutoString as(buffer);
-  delete [] buffer;
-  fputs(as, out);
+  PRUnichar* urlSpec = nsnull;
+  mInner->mURL->ToString(&urlSpec);
+  if (urlSpec) {
+    nsAutoString buffer(urlSpec);
+    delete [] urlSpec;
+    fputs(buffer, out);
+  }
 #endif
 
   if (mMedia) {
@@ -2387,10 +2391,11 @@ void CSSStyleSheetImpl::List(FILE* out, PRInt32 aIndent) const
     PRUint32 index = 0;
     PRUint32 count;
     mMedia->Count(&count);
+    nsAutoString  buffer;
     while (index < count) {
       nsIAtom* medium = (nsIAtom*)mMedia->ElementAt(index++);
-      medium->ToString(as);
-      fputs(as, out);
+      medium->ToString(buffer);
+      fputs(buffer, out);
       if (index < count) {
         fputs(", ", out);
       }
@@ -2749,10 +2754,12 @@ CSSStyleSheetImpl::GetHref(nsString& aHref)
 {
   if (mInner && mInner->mURL) {
 #ifdef NECKO
-    char* str;
+    char* str = nsnull;
     mInner->mURL->GetSpec(&str);
     aHref = str;
-    nsCRT::free(str);
+    if (str) {
+      nsCRT::free(str);
+    }
 #else
     PRUnichar* str;
     mInner->mURL->ToString(&str);
diff --git a/mozilla/layout/html/style/src/nsCSSStyleSheet.cpp b/mozilla/layout/html/style/src/nsCSSStyleSheet.cpp
index 16d8b66ac48..7adddb8cf4e 100644
--- a/mozilla/layout/html/style/src/nsCSSStyleSheet.cpp
+++ b/mozilla/layout/html/style/src/nsCSSStyleSheet.cpp
@@ -2370,16 +2370,20 @@ void CSSStyleSheetImpl::List(FILE* out, PRInt32 aIndent) const
 
   fputs("CSS Style Sheet: ", out);
 #ifdef NECKO
-  char* buffer;
-  nsresult rv = mInner->mURL->GetSpec(&buffer);
-  fputs(buffer, out);
-  nsCRT::free(buffer);
+  char* urlSpec = nsnull;
+  nsresult rv = mInner->mURL->GetSpec(&urlSpec);
+  if (urlSpec) {
+    fputs(urlSpec, out);
+    nsCRT::free(urlSpec);
+  }
 #else
-  PRUnichar* buffer;
-  mInner->mURL->ToString(&buffer);
-  nsAutoString as(buffer);
-  delete [] buffer;
-  fputs(as, out);
+  PRUnichar* urlSpec = nsnull;
+  mInner->mURL->ToString(&urlSpec);
+  if (urlSpec) {
+    nsAutoString buffer(urlSpec);
+    delete [] urlSpec;
+    fputs(buffer, out);
+  }
 #endif
 
   if (mMedia) {
@@ -2387,10 +2391,11 @@ void CSSStyleSheetImpl::List(FILE* out, PRInt32 aIndent) const
     PRUint32 index = 0;
     PRUint32 count;
     mMedia->Count(&count);
+    nsAutoString  buffer;
     while (index < count) {
       nsIAtom* medium = (nsIAtom*)mMedia->ElementAt(index++);
-      medium->ToString(as);
-      fputs(as, out);
+      medium->ToString(buffer);
+      fputs(buffer, out);
       if (index < count) {
         fputs(", ", out);
       }
@@ -2749,10 +2754,12 @@ CSSStyleSheetImpl::GetHref(nsString& aHref)
 {
   if (mInner && mInner->mURL) {
 #ifdef NECKO
-    char* str;
+    char* str = nsnull;
     mInner->mURL->GetSpec(&str);
     aHref = str;
-    nsCRT::free(str);
+    if (str) {
+      nsCRT::free(str);
+    }
 #else
     PRUnichar* str;
     mInner->mURL->ToString(&str);
diff --git a/mozilla/layout/style/nsCSSStyleSheet.cpp b/mozilla/layout/style/nsCSSStyleSheet.cpp
index 16d8b66ac48..7adddb8cf4e 100644
--- a/mozilla/layout/style/nsCSSStyleSheet.cpp
+++ b/mozilla/layout/style/nsCSSStyleSheet.cpp
@@ -2370,16 +2370,20 @@ void CSSStyleSheetImpl::List(FILE* out, PRInt32 aIndent) const
 
   fputs("CSS Style Sheet: ", out);
 #ifdef NECKO
-  char* buffer;
-  nsresult rv = mInner->mURL->GetSpec(&buffer);
-  fputs(buffer, out);
-  nsCRT::free(buffer);
+  char* urlSpec = nsnull;
+  nsresult rv = mInner->mURL->GetSpec(&urlSpec);
+  if (urlSpec) {
+    fputs(urlSpec, out);
+    nsCRT::free(urlSpec);
+  }
 #else
-  PRUnichar* buffer;
-  mInner->mURL->ToString(&buffer);
-  nsAutoString as(buffer);
-  delete [] buffer;
-  fputs(as, out);
+  PRUnichar* urlSpec = nsnull;
+  mInner->mURL->ToString(&urlSpec);
+  if (urlSpec) {
+    nsAutoString buffer(urlSpec);
+    delete [] urlSpec;
+    fputs(buffer, out);
+  }
 #endif
 
   if (mMedia) {
@@ -2387,10 +2391,11 @@ void CSSStyleSheetImpl::List(FILE* out, PRInt32 aIndent) const
     PRUint32 index = 0;
     PRUint32 count;
     mMedia->Count(&count);
+    nsAutoString  buffer;
     while (index < count) {
       nsIAtom* medium = (nsIAtom*)mMedia->ElementAt(index++);
-      medium->ToString(as);
-      fputs(as, out);
+      medium->ToString(buffer);
+      fputs(buffer, out);
       if (index < count) {
         fputs(", ", out);
       }
@@ -2749,10 +2754,12 @@ CSSStyleSheetImpl::GetHref(nsString& aHref)
 {
   if (mInner && mInner->mURL) {
 #ifdef NECKO
-    char* str;
+    char* str = nsnull;
     mInner->mURL->GetSpec(&str);
     aHref = str;
-    nsCRT::free(str);
+    if (str) {
+      nsCRT::free(str);
+    }
 #else
     PRUnichar* str;
     mInner->mURL->ToString(&str);