Use snprintf instead of sprintf everywhere in JavaScriptCore.
Created attachment 60417 [details] Patch
Comment on attachment 60417 [details] Patch r=me
Landed in r62414.
Comment on attachment 60417 [details] Patch JSGlobalObjectFunctions.cpp includes <string.h>, but now that it's using snprintf I think it needs to include StringExtras.h instead to avoid breaking the Windows build. review+ if you fix that. Also: UString::from(long long) uses snprintf, and for some reason passes sizeof(minBuf) - 1. WTF is up with that? And: UString.cpp seems to include StringExtras.h twice and also includes <string.h> unnecessarily since StringExtras.h includes that.