pcp
[Top] [All Lists]

Re: patch pcp-gui fix pmchart view save dialog

To: Mark Goodwin <goodwinos@xxxxxxxxx>
Subject: Re: patch pcp-gui fix pmchart view save dialog
From: fche@xxxxxxxxxx (Frank Ch. Eigler)
Date: Wed, 10 Apr 2013 19:31:01 -0400
Cc: pcp <pcp@xxxxxxxxxxx>
Delivered-to: pcp@xxxxxxxxxxx
In-reply-to: <516562BE.5040401@xxxxxxxxx> (Mark Goodwin's message of "Wed, 10 Apr 2013 23:01:50 +1000")
References: <516562BE.5040401@xxxxxxxxx>
User-agent: Gnus/5.1008 (Gnus v5.10.8) Emacs/21.4 (gnu/linux)
Mark Goodwin <goodwinos@xxxxxxxxx> writes:

> The pmchart view save dialog uses the wrong pointer value
> when saving a view for metrics with a singular instance
> domain.

My earlier ("Quite") patch fixes this more properly.  Can someone
review/merge it please?

>       saveInsts = cp->metric(m)->explicitInsts();
> +        p = (char *)(const char *)cp->name(m).toAscii();
>       if (saveInsts) {
> -         p = (char *)(const char *)cp->name(m).toAscii();

Constructs of this form are not valid, as the toAscii object
goes out of scope, making the p pointer invalid.

- FChE

<Prev in Thread] Current Thread [Next in Thread>