Bug 190793 - Style queue exception on some security bugs
Summary: Style queue exception on some security bugs
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Aakash Jain
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-10-22 06:55 PDT by Aakash Jain
Modified: 2018-12-07 10:50 PST (History)
8 users (show)

See Also:


Attachments
Proposed patch (1.38 KB, patch)
2018-12-07 01:28 PST, Aakash Jain
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews125 for ios-simulator-wk2 (2.40 MB, application/zip)
2018-12-07 03:18 PST, EWS Watchlist
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Aakash Jain 2018-10-22 06:55:14 PDT
Noticed following logs on the webkit-misc bot:

2018-10-22 04:42:29,914 - Failed to run "['/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-misc-style', 'apply-watchlist-local', 190625]" exit_code: 1 cwd: /Volumes/Data/StyleQueue/WebKit

Logging in as ews@webkit.org...
Fetching: https://bugs.webkit.org/show_bug.cgi?id=190625&ctype=xml&excludefield=attachmentdata
You don't have permission to view this bug.
Traceback (most recent call last):
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkit-patch", line 84, in <module>
    main()
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkit-patch", line 79, in main
    WebKitPatch(os.path.abspath(__file__)).main()
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/multicommandtool.py", line 305, in main
    result = command.check_arguments_and_execute(options, args, self)
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/multicommandtool.py", line 123, in check_arguments_and_execute
    return self.execute(options, args, tool) or 0
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/commands/abstractsequencedcommand.py", line 55, in execute
    self._sequence.run_and_handle_errors(tool, options, state)
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/commands/stepsequence.py", line 73, in run_and_handle_errors
    self._run(tool, options, state)
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/commands/stepsequence.py", line 67, in _run
    step(tool, options).run(state)
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/steps/applywatchlist.py", line 56, in run 
    cc_emails = set(cc_emails).difference(bug.cc_emails())
AttributeError: 'NoneType' object has no attribute 'cc_emails

webkit-misc:style-queue-logs buildbot$ grep "object has no attribute" *.log | wc -l
     435
Comment 1 Aakash Jain 2018-12-07 01:15:47 PST
This has been happening multiple times daily.

[~] grep "object has no attribute" *.log
style-queue_2018-11-06_04-11.log:AttributeError: 'NoneType' object has no attribute 'cc_emails'
style-queue_2018-11-06_04-11.log:AttributeError: 'NoneType' object has no attribute 'cc_emails'
style-queue_2018-11-06_04-11.log:AttributeError: 'NoneType' object has no attribute 'cc_emails'
style-queue_2018-11-06_04-11.log:AttributeError: 'NoneType' object has no attribute 'cc_emails'
style-queue_2018-11-06_10-11.log:AttributeError: 'NoneType' object has no attribute 'cc_emails'
style-queue_2018-11-06_10-11.log:AttributeError: 'NoneType' object has no attribute 'cc_emails'
style-queue_2018-11-06_10-11.log:AttributeError: 'NoneType' object has no attribute 'cc_emails'
style-queue_2018-11-06_10-11.log:AttributeError: 'NoneType' object has no attribute 'cc_emails'
...
...
style-queue_2018-12-05_17-12.log:AttributeError: 'NoneType' object has no attribute 'cc_emails'
style-queue_2018-12-06_01-12.log:AttributeError: 'NoneType' object has no attribute 'cc_emails'
style-queue_2018-12-06_01-12.log:AttributeError: 'NoneType' object has no attribute 'cc_emails'
style-queue_2018-12-06_09-12.log:AttributeError: 'NoneType' object has no attribute 'cc_emails'
style-queue_2018-12-06_09-12.log:AttributeError: 'NoneType' object has no attribute 'cc_emails'
style-queue_2018-12-06_09-12.log:AttributeError: 'NoneType' object has no attribute 'cc_emails'
style-queue_2018-12-06_16-12.log:AttributeError: 'NoneType' object has no attribute 'cc_emails'
style-queue_2018-12-06_16-12.log:AttributeError: 'NoneType' object has no attribute 'cc_emails'
style-queue_2018-12-07_00-12.log:AttributeError: 'NoneType' object has no attribute 'cc_emails'
style-queue_2018-12-07_00-12.log:AttributeError: 'NoneType' object has no attribute 'cc_emails'
style-queue_2018-12-07_00-12.log:AttributeError: 'NoneType' object has no attribute 'cc_emails'
Comment 2 Aakash Jain 2018-12-07 01:28:28 PST
Created attachment 356788 [details]
Proposed patch
Comment 3 EWS Watchlist 2018-12-07 01:33:01 PST
Attachment 356788 [details] did not pass style-queue:


ERROR: Tools/ChangeLog:3:  Please consider whether the use of security-sensitive phrasing could help someone exploit WebKit: security bug  [changelog/unwantedsecurityterms] [3]
Total errors found: 1 in 2 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 4 EWS Watchlist 2018-12-07 03:18:50 PST
Comment on attachment 356788 [details]
Proposed patch

Attachment 356788 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: https://webkit-queues.webkit.org/results/10302526

New failing tests:
imported/w3c/web-platform-tests/service-workers/service-worker/register-closed-window.https.html
Comment 5 EWS Watchlist 2018-12-07 03:18:52 PST
Created attachment 356795 [details]
Archive of layout-test-results from ews125 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews125  Port: ios-simulator-wk2  Platform: Mac OS X 10.13.6
Comment 6 WebKit Commit Bot 2018-12-07 10:49:03 PST
Comment on attachment 356788 [details]
Proposed patch

Clearing flags on attachment: 356788

Committed r238956: <https://trac.webkit.org/changeset/238956>
Comment 7 WebKit Commit Bot 2018-12-07 10:49:04 PST
All reviewed patches have been landed.  Closing bug.
Comment 8 Radar WebKit Bug Importer 2018-12-07 10:50:35 PST
<rdar://problem/46558454>