----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129994/#review102764 -----------------------------------------------------------
Ship it! Ship It! - David Edmundson On March 8, 2017, 2:17 a.m., Leslie Zhai wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/129994/ > ----------------------------------------------------------- > > (Updated March 8, 2017, 2:17 a.m.) > > > Review request for Plasma, Kai Uwe Broulik and David Edmundson. > > > Repository: plasma-workspace > > > Description > ------- > > Hi KDE developer, > > Bug reported by the clang static analyzer. > > Description: Called C++ object pointer is null > File: plasma-workspace/applets/appmenu/lib/appmenuapplet.cpp > Line: 167 > > ``` > 153 void AppMenuApplet::trigger(QQuickItem *ctx, int idx) > 154 { > 155 if (m_currentIndex == idx) { > > Taking false branch > ? > 156 return; > 157 } > 158 > 159 QMenu *actionMenu = createMenu(idx); > 160 if (actionMenu) { > > ? > Assuming 'actionMenu' is non-null > ? > > ? > Taking true branch > ? > 161 > 162 if (ctx && ctx->window() && ctx->window()->mouseGrabberItem()) { > 4 > ? > Assuming 'ctx' is null > ? > 163 // FIXME event forge thing enters press and hold move mode > :/ > 164 ctx->window()->mouseGrabberItem()->ungrabMouse(); > 165 } > 166 > 167 const auto &geo = > ctx->window()->screen()->availableVirtualGeometry(); > 5 > ? > Called C++ object pointer is null > ``` > > So I simply add some checker for ctx, ctx->window() and > ctx->window()->screen() pointers. > > Regards, > Leslie Zhai > > > Diffs > ----- > > applets/appmenu/lib/appmenuapplet.cpp 9aaf5cc6 > > Diff: https://git.reviewboard.kde.org/r/129994/diff/ > > > Testing > ------- > > > Thanks, > > Leslie Zhai > >
