Bug 682710 – shell-app: Fix a memory leak in the running state

After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.

Bug 682710 - shell-app: Fix a memory leak in the running state


Summary:	shell-app: Fix a memory leak in the running state

Reported:	2012-08-26 01:24 UTC by Jasper St. Pierre (not reading bugmail)
Modified:	2012-08-26 14:56 UTC

See Also:
GNOME target:	---
GNOME version:	---

Attachments
shell-app: Fix a memory leak in the running state (824 bytes, patch) 2012-08-26 01:24 UTC, Jasper St. Pierre (not reading bugmail)	committed	Details \| Review

Description Jasper St. Pierre (not reading bugmail) 2012-08-26 01:24:49 UTC

See patch. Spotted by darkxzt on IRC.

Comment 1 Jasper St. Pierre (not reading bugmail) 2012-08-26 01:24:51 UTC

Created attachment 222445 [details] [review]
shell-app: Fix a memory leak in the running state

We need to two strings that we allocated here.

Comment 2 drago01 2012-08-26 04:54:33 UTC

Review of attachment 222445 [details] [review]:

"We need to two strings that we allocated here."

"need to free" otherwise looks good.

Comment 3 Jasper St. Pierre (not reading bugmail) 2012-08-26 14:56:23 UTC

Attachment 222445 [details] pushed as 785ab81 - shell-app: Fix a memory leak in the running state