Mirrors_Framework/mujs
1
0
Fork 0
mirror of https://github.com/ccxvii/mujs.git synced 2026-02-06 01:41:37 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix bug in Ap_sort where tostring values were put back into the array.

Browse Source
This commit is contained in:
Tor Andersson
2018-04-27 11:11:50 +02:00
parent 57b698f0b2
commit 780b14b5b7
1 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
jsarray.c
View File

@@ -274,9 +274,15 @@ static int compare(js_State *J, int x, int y, int *hasx, int *hasy, int hasfn)
return c;
}
/* Ap_sort expects the original values to remain on the stack,
* but because js_tostring may mutate the stack slot, make a copy first. */
js_copy(J, -2);
js_copy(J, -2);
sx = js_tostring(J, -2);
sy = js_tostring(J, -1);
return strcmp(sx, sy);
c = strcmp(sx, sy);
js_pop(J, 2);
return c;
}
if (*hasx) return -1;