Bug 56390 - Valgrind error due to uninitialized PluginLayerChromium::m_textureId
Summary: Valgrind error due to uninitialized PluginLayerChromium::m_textureId
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Plug-ins (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Alok Priyadarshi
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-03-15 11:14 PDT by Alok Priyadarshi
Modified: 2011-03-15 13:17 PDT (History)
3 users (show)

See Also:


Attachments
proposed patch (4.41 KB, patch)
2011-03-15 11:41 PDT, Alok Priyadarshi
jamesr: review+
commit-queue: commit-queue-
Details | Formatted Diff | Diff
proposed patch (7.07 KB, patch)
2011-03-15 11:56 PDT, Alok Priyadarshi
no flags Details | Formatted Diff | Diff
proposed patch (3.54 KB, patch)
2011-03-15 12:05 PDT, Alok Priyadarshi
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alok Priyadarshi 2011-03-15 11:14:45 PDT
Valgrind generates the following error due to uninitialized PluginLayerChromium::m_textureId variable:

UninitCondition
Conditional jump or move depends on uninitialised value(s)
  WebKit::WebPluginContainerImpl::platformLayer() const (third_party/WebKit/Source/WebKit/chromium/src/WebPluginContainerImpl.cpp:446)
  WebCore::RenderEmbeddedObject::allowsAcceleratedCompositing() const (third_party/WebKit/Source/WebCore/rendering/RenderEmbeddedObject.cpp:107)
  WebCore::RenderEmbeddedObject::requiresLayer() const (third_party/WebKit/Source/WebCore/rendering/RenderEmbeddedObject.cpp:102)
  WebCore::RenderObject::adjustStyleDifference(WebCore::StyleDifference, unsigned int) const (third_party/WebKit/Source/WebCore/rendering/RenderObject.cpp:1708)
  WebCore::RenderObject::setStyle(WTF::PassRefPtr<WebCore::RenderStyle>) (third_party/WebKit/Source/WebCore/rendering/RenderObject.cpp:1738)
  WebCore::RenderObject::setAnimatableStyle(WTF::PassRefPtr<WebCore::RenderStyle>) (third_party/WebKit/Source/WebCore/rendering/RenderObject.cpp:1676)
  WebCore::Node::setRenderStyle(WTF::PassRefPtr<WebCore::RenderStyle>) (third_party/WebKit/Source/WebCore/dom/Node.cpp:1482)
  WebCore::Element::recalcStyle(WebCore::Node::StyleChange) (third_party/WebKit/Source/WebCore/dom/Element.cpp:1089)
  WebCore::HTMLPlugInImageElement::recalcStyle(WebCore::Node::StyleChange) (third_party/WebKit/Source/WebCore/html/HTMLPlugInImageElement.cpp:129)
  WebCore::Element::recalcStyle(WebCore::Node::StyleChange) (third_party/WebKit/Source/WebCore/dom/Element.cpp:1113)
  WebCore::Element::recalcStyle(WebCore::Node::StyleChange) (third_party/WebKit/Source/WebCore/dom/Element.cpp:1113)
  WebCore::Document::recalcStyle(WebCore::Node::StyleChange) (third_party/WebKit/Source/WebCore/dom/Document.cpp:1526)
  WebCore::Document::updateStyleIfNeeded() (third_party/WebKit/Source/WebCore/dom/Document.cpp:1576)
  WebCore::Document::styleRecalcTimerFired(WebCore::Timer<WebCore::Document>*) (third_party/WebKit/Source/WebCore/dom/Document.cpp:1477)
Comment 1 Alok Priyadarshi 2011-03-15 11:41:44 PDT
Created attachment 85834 [details]
proposed patch

Initialized member variable and also addressed your comment about function naming.
Comment 2 WebKit Commit Bot 2011-03-15 11:52:50 PDT
Comment on attachment 85834 [details]
proposed patch

Rejecting attachment 85834 [details] from commit-queue.

Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=cr-jail-3', 'apply-..." exit_code: 2

Last 500 characters of output:
ching file Source/WebKit/chromium/public/WebPlugin.h
Hunk #1 FAILED at 64.
1 out of 1 hunk FAILED -- saving rejects to file Source/WebKit/chromium/public/WebPlugin.h.rej
patching file Source/WebKit/chromium/src/WebPluginContainerImpl.cpp
Hunk #2 FAILED at 443.
1 out of 2 hunks FAILED -- saving rejects to file Source/WebKit/chromium/src/WebPluginContainerImpl.cpp.rej

Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--reviewer', u'James Robinson', u'--f..." exit_code: 1

Full output: http://queues.webkit.org/results/8178621
Comment 3 Alok Priyadarshi 2011-03-15 11:56:27 PDT
Created attachment 85837 [details]
proposed patch

Sorry the last patch also included changes from an earlier CL. Now from the correct client.
Comment 4 WebKit Review Bot 2011-03-15 11:57:30 PDT
Attachment 85837 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1

Source/WebCore/platform/graphics/chromium/LayerRendererChromium.h:72:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/WebCore/platform/graphics/chromium/LayerRendererChromium.h:73:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/WebCore/platform/graphics/chromium/LayerRendererChromium.h:75:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/WebCore/platform/graphics/chromium/LayerRendererChromium.h:76:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/WebCore/platform/graphics/chromium/LayerRendererChromium.h:77:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/WebCore/platform/graphics/chromium/LayerRendererChromium.h:80:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/WebCore/platform/graphics/chromium/LayerRendererChromium.h:81:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Total errors found: 7 in 7 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 5 James Robinson 2011-03-15 12:03:51 PDT
Comment on attachment 85837 [details]
proposed patch

wrong patch
Comment 6 Alok Priyadarshi 2011-03-15 12:05:45 PDT
Created attachment 85838 [details]
proposed patch
Comment 7 WebKit Commit Bot 2011-03-15 13:14:14 PDT
The commit-queue encountered the following flaky tests while processing attachment 85838 [details]:

transitions/interrupted-accelerated-transition.html bug 56242 (authors: simon.fraser@apple.com and tonyg@chromium.org)
The commit-queue is continuing to process your patch.
Comment 8 WebKit Commit Bot 2011-03-15 13:17:39 PDT
Comment on attachment 85838 [details]
proposed patch

Clearing flags on attachment: 85838

Committed r81169: <http://trac.webkit.org/changeset/81169>
Comment 9 WebKit Commit Bot 2011-03-15 13:17:44 PDT
All reviewed patches have been landed.  Closing bug.