My favorites | Sign in
Project Home Downloads Wiki Issues Code Search
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 100213: Triage PVS warnings
4 people starred this issue and may be notified of changes. Back to list
Status:  Fixed
Owner:  thakis@google.com
Closed:  Aug 2012
Cc:  rsleevi@chromium.org, cev...@chromium.org

Restricted
  • Only users with Commit permission may comment.


Sign in to add a comment
 
Reported by thakis@google.com, Oct 13, 2011
http://www.viva64.com/en/b/0113/

Much of that we already know about, but we should triage it none the less.
Oct 13, 2011
#1 thakis@google.com
N1: I left a comment on http://codereview.chromium.org/7258002
N4: That's  issue 83234 
Oct 13, 2011
#2 dmichael@google.com
I filed for N1: crbug.com/100215
Oct 13, 2011
#5 thakis@google.com
N12: http://codereview.chromium.org/8273009
Blockedon: 83234
Oct 13, 2011
#6 thakis@google.com
(No comment was entered for this change.)
Blockedon: 100215
Oct 13, 2011
#7 thakis@chromium.org
(No comment was entered for this change.)
Blockedon: 100257
Oct 13, 2011
#8 rsleevi@chromium.org
N1:  Issue 100215 
N2:  Issue 100257 
N3:  Issue 84136 
N4:  Issue 83234 
N5:  Issue webrtc:110  ( https://code.google.com/p/webrtc/issues/detail?id=110 )
N6: https://bugs.freedesktop.org/show_bug.cgi?id=41768 (fixed in git 6fc6d5a09e36c56f4debd17bbbed15f732403619 )
N7:  Issue webrtc:111  ( https://code.google.com/p/webrtc/issues/detail?id=111 )
N8: http://codereview.chromium.org/8273023
N9:  Issue webrtc:112  ( https://code.google.com/p/webrtc/issues/detail?id=112 )
N10: Fixed upstream already. Adding Chris Evans, as he's been working on security analysis, to know what, if any, impact this might have.
N11: Not yet filed, as I need to analyze a bit more to figure out what's expected.
N12: http://codereview.chromium.org/8273009
Cc: cev...@chromium.org
Blockedon: 84136 webrtc:110 webrtc:111 webrtc:112
Oct 13, 2011
#9 rsleevi@chromium.org
N10 is filed as https://bugs.webkit.org/show_bug.cgi?id=70074 , cc'd rniwa who originally introduced it
Oct 15, 2011
#10 thakis@google.com
 Issue 100206  has been merged into this issue.
Oct 15, 2011
#11 rsleevi@chromium.org
Sorry, N11 was the webkit bug, but has been fixed and is pending roll of WK.
Oct 16, 2011
#12 bugdro...@chromium.org
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=105763

------------------------------------------------------------------------
r105763 | rsleevi@chromium.org | Sun Oct 16 20:29:01 PDT 2011

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/content_settings/content_settings_origin_identifier_value_map.cc?r1=105763&r2=105762&pathrev=105763

Remove an unnecessary comparison in OriginIdentifierValueMap::EntryMapKey::operator<

Originally reported by viva64 using PVS Studio ( http://www.viva64.com/en/b/0113/ Fragment N8)

BUG=100213
TEST=none


Review URL: http://codereview.chromium.org/8273023
------------------------------------------------------------------------
Mar 6, 2012
#13 lafo...@google.com
Available + Owner == Default to Assigned
Status: Assigned
Aug 23, 2012
#14 thakis@chromium.org
(No comment was entered for this change.)
Status: Fixed
Blockedon: -chromium:83234 -chromium:84136 -chromium:100215 -chromium:100257 -webrtc:110 -webrtc:111 -webrtc:112 chromium:83234 chromium:84136 chromium:100215 chromium:100257 webrtc:110 webrtc:111 webrtc:112
Oct 13, 2012
#15 bugdro...@chromium.org
This issue has been closed for some time. No one will pay attention to new comments.
If you are seeing this bug or have new data, please click New Issue to start a new bug.
Labels: Restrict-AddIssueComment-Commit
Mar 9, 2013
#16 bugdro...@chromium.org
(No comment was entered for this change.)
Labels: -Area-Build Build
Sign in to add a comment

Powered by Google Project Hosting