qbit/xenocara
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Plug two memory leaks. Also get rid of a variable that is no longer

Browse Source 
necessary.

OK okan@
This commit is contained in:
tim 2020-03-13 20:49:13 +00:00
parent fb24e5a8dd
commit 8990b0be5d
1 changed files with 10 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
app/cwm/client.c
View File

@ -15,7 +15,7 @@
* ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
* OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE. * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
* *
* $OpenBSD: client.c,v 1.258 2020/02/28 13:38:35 okan Exp $ * $OpenBSD: client.c,v 1.259 2020/03/13 20:49:13 tim Exp $
*/ */
#include <sys/types.h> #include <sys/types.h>
@ -668,21 +668,23 @@ void
client_set_name(struct client_ctx *cc) client_set_name(struct client_ctx *cc)
{ {
struct winname *wn; struct winname *wn;
char *newname;
int i = 0; int i = 0;
if (!xu_get_strprop(cc->win, ewmh[_NET_WM_NAME], &newname)) free(cc->name);
if (!xu_get_strprop(cc->win, XA_WM_NAME, &newname)) if (!xu_get_strprop(cc->win, ewmh[_NET_WM_NAME], &cc->name))
newname = xstrdup(""); if (!xu_get_strprop(cc->win, XA_WM_NAME, &cc->name))
cc->name = xstrdup("");
TAILQ_FOREACH(wn, &cc->nameq, entry) { TAILQ_FOREACH(wn, &cc->nameq, entry) {
if (strcmp(wn->name, newname) == 0) if (strcmp(wn->name, cc->name) == 0) {
TAILQ_REMOVE(&cc->nameq, wn, entry); TAILQ_REMOVE(&cc->nameq, wn, entry);
free(wn->name);
free(wn);
}
i++; i++;
} }
cc->name = newname;
wn = xmalloc(sizeof(*wn)); wn = xmalloc(sizeof(*wn));
wn->name = xstrdup(newname); wn->name = xstrdup(cc->name);
TAILQ_INSERT_TAIL(&cc->nameq, wn, entry); TAILQ_INSERT_TAIL(&cc->nameq, wn, entry);
/* Garbage collection. */ /* Garbage collection. */