Review Request 129994: Fix AppMenuApplet Called C++ object pointer is null

Leslie Zhai xiangzhai83 at gmail.com
Thu Mar 9 01:43:32 UTC 2017


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129994/
-----------------------------------------------------------

(Updated March 8, 2017, 6:43 p.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma, Kai Uwe Broulik and David Edmundson.


Changes
-------

Submitted with commit 2b7ef97f8104db6212e2a75d2578c6edca96e05a by Leslie Zhai to branch master.


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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170309/e57e9d50/attachment.html>


More information about the Plasma-devel mailing list