Bug 48284 – Valgrind failure in GraphicsContext3DInternal::reshape

• Component: WebGL
• Status: RESOLVED FIXED

[Kenneth Russell]

There is a Valgrind failure reported in GraphicsContext3DInternal::reshape caused by uninitialized members m_cachedWidth and m_cachedHeight in WebGraphicsContext3DDefaultImpl. The stack trace:

14:19:56 memcheck_analyze.py [ERROR] FAIL! There were 1 errors: 
14:19:56 memcheck_analyze.py [ERROR] Command: /b/slave/webkit-rel-linux-valgrind-layout/build/src/sconsbuild/Release/test_shell --pixel-tests=/b/slave/webkit-rel-linux-valgrind-layout/build/src/webkit/Release/layout-test-results/png_result1.png --layout-tests
UninitCondition
Conditional jump or move depends on uninitialised value(s)
  WebCore::GraphicsContext3DInternal::reshape(int, int) (third_party/WebKit/WebKit/chromium/src/GraphicsContext3DChromium.cpp:229)
  WebCore::GraphicsContext3D::reshape(int, int) (third_party/WebKit/WebKit/chromium/src/GraphicsContext3DChromium.cpp:841)
  WebCore::WebGLRenderingContext::WebGLRenderingContext(WebCore::HTMLCanvasElement*, WTF::PassRefPtr<WebCore::GraphicsContext3D>) (third_party/WebKit/WebCore/html/canvas/WebGLRenderingContext.cpp:139)
  WebCore::WebGLRenderingContext::create(WebCore::HTMLCanvasElement*, WebCore::WebGLContextAttributes*) (third_party/WebKit/WebCore/html/canvas/WebGLRenderingContext.cpp:96)
  WebCore::HTMLCanvasElement::getContext(WTF::String const&, WebCore::CanvasContextAttributes*) (third_party/WebKit/WebCore/html/HTMLCanvasElement.cpp:173)
  WebCore::V8HTMLCanvasElement::getContextCallback(v8::Arguments const&) (third_party/WebKit/WebCore/bindings/v8/custom/V8HTMLCanvasElementCustom.cpp:82)
  v8::internal::Object* v8::internal::HandleApiCallHelper<false>(v8::internal::(anonymous namespace)::BuiltinArguments<(v8::internal::BuiltinExtraArguments)1>) (v8/src/builtins.cc:983)
  0x5C6422D ()
  0x5C7C271 ()
  0x5C6609E ()
  0x5C7B4C9 ()
  0x5C75058 ()
  0x5C65421 ()
  v8::internal::Invoke(bool, v8::internal::Handle<v8::internal::JSFunction>, v8::internal::Handle<v8::internal::Object>, int, v8::internal::Object***, bool*) (v8/src/execution.cc:94)
  0x4E8E83B ()

[Kenneth Russell]

Created attachment 71828 [details]
Patch

[Andreas Kling]

Comment on attachment 71828 [details]
Patch

LGTM.

[Kenneth Russell]

Committed r70534: <http://trac.webkit.org/changeset/70534>