| 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> |
|---|---|---|
| ||
| Previous by Date: | Re: [pcp] qa/713 certificate issue, Ken McDonell |
|---|---|
| Next by Date: | pcp updates, Ken McDonell |
| Previous by Thread: | patch pcp-gui fix pmchart view save dialog, Mark Goodwin |
| Next by Thread: | Re: [pcp] patch pcp-gui fix pmchart view save dialog, Mark Goodwin |
| Indexes: | [Date] [Thread] [Top] [All Lists] |