New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 640066 link

Starred by 3 users
Status: Fixed
Owner:
thestig@chromium.org
Closed: Aug 2016
Cc:
thestig@chromium.org
█ groby@chromium.org
dbeam@chromium.org
pfeldman@chromium.org
mmenke@chromium.org
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug



Sign in to add a comment

Use-of-uninitialized-value in GetResponseInfo()

Project Member Reported by thestig@chromium.org, Aug 23 2016 
https://build.chromium.org/p/chromium.memory.full/builders/Linux%20ChromeOS%20MSan%20Tests/builds/522

UrlDataManagerBackendTest.ChromeNetworkErrorPageRequestFailed (run #1):
[ RUN      ] UrlDataManagerBackendTest.ChromeNetworkErrorPageRequestFailed
==12253==WARNING: MemorySanitizer: use-of-uninitialized-value
    #0 0x9d66bfe in GetResponseInfo content/browser/webui/url_data_manager_backend.cc:369:7
    #1 0xdd67f64 in ?? net/url_request/url_request_job.cc:598:3
    #2 0x9d65215 in ?? content/browser/webui/url_data_manager_backend.cc:444:5

  Uninitialized value was created by a heap allocation
    #0 0x6d5902 in operator new(unsigned long) ??:0
    #1 0x9d6e5d8 in MaybeCreateJob content/browser/webui/url_data_manager_backend.cc:546:12
    #2 0xdd6a2df in MaybeCreateJobWithProtocolHandler net/url_request/url_request_job_factory_impl.cc:60:22
    #3 0xdd6cb13 in CreateJob net/url_request/url_request_job_manager.cc:71:37
    #4 0xdd4a650 in Start net/url_request/url_request.cc:536:49
    #5 0x3a3a57a in TestBody content/browser/webui/url_data_manager_backend_unittest.cc:133:18
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 23 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/922d34c91c451381eb1ce42f44ee1efb7b33965d

commit 922d34c91c451381eb1ce42f44ee1efb7b33965d
Author: thestig <thestig@chromium.org>
Date: Tue Aug 23 03:40:52 2016

Initialize a member in URLRequestChromeJob.

Clean up some nits.

BUG= 640066 

Review-Url: https://codereview.chromium.org/2268903002
Cr-Commit-Position: refs/heads/master@{#413657}

[modify] https://crrev.com/922d34c91c451381eb1ce42f44ee1efb7b33965d/content/browser/webui/url_data_manager_backend.cc

Comment 2 by thestig@chromium.org, Aug 23 2016

Status: Fixed (was: Assigned)

Comment 3 by thestig@chromium.org, Aug 23 2016

Cc: thestig@chromium.org pfeldman@chromium.org mmenke@chromium.org
 Issue 639810  has been merged into this issue.
►

Sign in to add a comment