Bug 676187 - Ensure fileContents are freed, fixed a memory leak in toolkit/mozapps/readstrings/readstrings.cpp. r=bbondy

This commit is contained in:
Hugo Tavares Reis 2011-12-14 23:39:31 -08:00
parent 4c7782beb7
commit 69d0e0619c

View File

@ -61,6 +61,15 @@ private:
FILE *fp_;
};
class AutoCharArray {
public:
AutoCharArray(size_t len) { ptr_ = new char[len]; }
~AutoCharArray() { delete[] ptr_; }
operator char *() { return ptr_; }
private:
char *ptr_;
};
static const char kNL[] = "\r\n";
static const char kEquals[] = "=";
static const char kWhitespace[] = " \t";
@ -167,7 +176,7 @@ ReadStrings(const NS_tchar *path,
return READ_ERROR;
size_t flen = size_t(len);
char *fileContents = new char[flen + 1];
AutoCharArray fileContents(flen + 1);
if (!fileContents)
return MEM_ERROR;