Bug 199532 - [GTK] ViewGestureController constructor leaves m_direction uninitialized
Summary: [GTK] ViewGestureController constructor leaves m_direction uninitialized
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Nightly Build
Hardware: PC Linux
: P2 Normal
Assignee: Michael Catanzaro
URL:
Keywords:
Depends on:
Blocks: 104114
  Show dependency treegraph
 
Reported: 2019-07-05 14:54 PDT by Michael Catanzaro
Modified: 2019-07-08 01:17 PDT (History)
5 users (show)

See Also:


Attachments
Patch (1.47 KB, patch)
2019-07-05 14:55 PDT, Michael Catanzaro
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Catanzaro 2019-07-05 14:54:36 PDT
ViewGestureController constructor leaves m_direction uninitialized

This is harmless because it should never be read until after it's initialized, but still not robust.
Comment 1 Michael Catanzaro 2019-07-05 14:55:59 PDT
Created attachment 373541 [details]
Patch
Comment 2 Michael Catanzaro 2019-07-05 15:01:38 PDT
P.S. Exalm, you should add yourself to Tools/Scripts/webkitpy/common/config/contributors.json so you'll be easier to CC ;)
Comment 3 Alice Mikhaylenko 2019-07-05 15:56:51 PDT
Oops. :) But yeah, it's always initialized before it's used.
Comment 4 WebKit Commit Bot 2019-07-08 01:17:30 PDT
Comment on attachment 373541 [details]
Patch

Clearing flags on attachment: 373541

Committed r247203: <https://trac.webkit.org/changeset/247203>
Comment 5 WebKit Commit Bot 2019-07-08 01:17:31 PDT
All reviewed patches have been landed.  Closing bug.