Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 29404594: Noissue - [emscripten] Replace sprintf() usage by safe alternatives (Closed)

Created:
April 6, 2017, 8:40 a.m. by Wladimir Palant
Modified:
April 6, 2017, 6:29 p.m.
Reviewers:
hub
CC:
Felix Dahlke
Base URL:
https://hg.adblockplus.org/adblockpluscore
Visibility:
Public.

Description

Noissue - [emscripten] Replace sprintf() usage by safe alternatives

Patch Set 1 #

Total comments: 2

Patch Set 2 : Implemented a simple number conversion method #

Unified diffs Side-by-side diffs Delta from patch set Stats (+37 lines, -19 lines) Patch
M compiled/String.h View 1 2 chunks +31 lines, -1 line 0 comments Download
M compiled/bindings.ipp View 1 1 chunk +4 lines, -5 lines 0 comments Download
M compiled/filter/ActiveFilter.cpp View 1 2 chunks +2 lines, -13 lines 0 comments Download

Messages

Total messages: 4
Wladimir Palant
April 6, 2017, 8:40 a.m. (2017-04-06 08:40:25 UTC) #1
hub
important issue. https://codereview.adblockplus.org/29404594/diff/29404595/compiled/filter/ActiveFilter.cpp File compiled/filter/ActiveFilter.cpp (right): https://codereview.adblockplus.org/29404594/diff/29404595/compiled/filter/ActiveFilter.cpp#newcode33 compiled/filter/ActiveFilter.cpp:33: OwnedString result(len); From `man snprintf` > if ...
April 6, 2017, 10:31 a.m. (2017-04-06 10:31:23 UTC) #2
Wladimir Palant
Patch set 2 has been rebased and applies to current repository state. https://codereview.adblockplus.org/29404594/diff/29404595/compiled/filter/ActiveFilter.cpp File compiled/filter/ActiveFilter.cpp ...
April 6, 2017, 3:14 p.m. (2017-04-06 15:14:40 UTC) #3
hub
April 6, 2017, 4:08 p.m. (2017-04-06 16:08:02 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld