Bommels05/Mozilla
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix uninitialized variable use in an assertion (311583, r=igor).

Browse Source 
git-svn-id: svn://10.0.0.236/trunk@185772 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
brendan%mozilla.org
2005-12-07 22:42:06 +00:00
parent ab85088d77
commit 834e997e9f
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
mozilla/js/src/jsarray.c
View File

@@ -396,6 +396,10 @@ array_join_sub(JSContext *cx, JSObject *obj, enum ArrayToSringOp op,
he = js_EnterSharpObject(cx, obj, NULL, &chars);
if (!he)
return JS_FALSE;
#ifdef DEBUG
growth = (size_t) -1;
#endif
if (op == TO_SOURCE) {
if (IS_SHARP(he)) {
#if JS_HAS_SHARP_VARS
@@ -434,7 +438,6 @@ array_join_sub(JSContext *cx, JSObject *obj, enum ArrayToSringOp op,
JS_ASSERT(sep == NULL);
sepstr = NULL; /* indicates to use ", " as separator */
seplen = 2;
} else {
/*
* Free any sharp variable definition in chars. Normally, we would
@@ -586,7 +589,7 @@ array_join_sub(JSContext *cx, JSObject *obj, enum ArrayToSringOp op,
return JS_FALSE;
}
chars[nchars] = 0;
JS_ASSERT((nchars + 1) * sizeof(jschar) == growth);
JS_ASSERT(growth == (size_t)-1 || (nchars + 1) * sizeof(jschar) == growth);
str = js_NewString(cx, chars, nchars, 0);
if (!str) {
free(chars);