RESOLVED FIXED 171520
Stop using strcpy() in WebProcess::registerWithStateDumper() in WebProcessCocoa.mm
https://bugs.webkit.org/show_bug.cgi?id=171520
Summary Stop using strcpy() in WebProcess::registerWithStateDumper() in WebProcessCoc...
David Kilzer (:ddkilzer)
Reported 2017-05-01 15:49:24 PDT
$ ./Tools/Scripts/check-webkit-style --filter="-,+security/" Source/WebKit2 [GTK warnings elided] ERROR: Source/WebKit2/WebProcess/cocoa/WebProcessCocoa.mm:286: Almost always, snprintf is better than strcpy. [security/printf] [4] Total errors found: 1 in 2838 files Found via re-enabled checkers from Bug 171494.
Attachments
Patch v1 (1.62 KB, patch)
2017-05-01 15:53 PDT, David Kilzer (:ddkilzer)
no flags
Patch v2 (1.61 KB, patch)
2017-05-02 02:53 PDT, David Kilzer (:ddkilzer)
no flags
Archive of layout-test-results from ews122 for ios-simulator-wk2 (700.08 KB, application/zip)
2017-05-02 04:59 PDT, Build Bot
no flags
David Kilzer (:ddkilzer)
Comment 1 2017-05-01 15:53:29 PDT
Created attachment 308780 [details] Patch v1
David Kilzer (:ddkilzer)
Comment 2 2017-05-02 02:53:23 PDT
Created attachment 308812 [details] Patch v2
Build Bot
Comment 3 2017-05-02 04:59:13 PDT
Comment on attachment 308812 [details] Patch v2 Attachment 308812 [details] did not pass ios-sim-ews (ios-simulator-wk2): Output: http://webkit-queues.webkit.org/results/3657200 New failing tests: compositing/absolute-inside-out-of-view-fixed.html
Build Bot
Comment 4 2017-05-02 04:59:14 PDT
Created attachment 308816 [details] Archive of layout-test-results from ews122 for ios-simulator-wk2 The attached test failures were seen while running run-webkit-tests on the ios-sim-ews. Bot: ews122 Port: ios-simulator-wk2 Platform: Mac OS X 10.11.6
Chris Dumez
Comment 5 2017-05-02 13:52:56 PDT
Comment on attachment 308812 [details] Patch v2 r=me
WebKit Commit Bot
Comment 6 2017-05-02 14:25:02 PDT
Comment on attachment 308812 [details] Patch v2 Clearing flags on attachment: 308812 Committed r216095: <http://trac.webkit.org/changeset/216095>
WebKit Commit Bot
Comment 7 2017-05-02 14:25:03 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.