pcp
[Top] [All Lists]

pcp update: RHBZ1187588 - PM_CONTEXT_LOCAL crashes process when /etc/pcp

To: pcp developers <pcp@xxxxxxxxxxx>
Subject: pcp update: RHBZ1187588 - PM_CONTEXT_LOCAL crashes process when /etc/pcp.conf is unreadable
From: "Frank Ch. Eigler" <fche@xxxxxxxxxx>
Date: Thu, 26 Feb 2015 16:44:09 -0500
Cc: mvollmer@xxxxxxxxxx
Delivered-to: pcp@xxxxxxxxxxx
User-agent: Mutt/1.4.2.2i
Hi -

Please see pcpfans.git fche/rhbz1187588.

commit ce4c4b526b279e29e18b2f0805173be8ffe8b2b7
Author: Frank Ch. Eigler <fche@xxxxxxxxxx>
Date:   Thu Feb 26 16:34:01 2015 -0500

    RHBZ1187588: don't exit() from a failing pmGetConfig
    
    An eagle-eyed mvollmer noted the ominous warnings & code for
    pmGetConfig(), wherein a missing pcp.conf file was deemed a "FATAL PCP
    ERROR" and resulted in a process exit().  This sort of error handling
    is not appropriate within a library, as the application is better
    equipped to judge the severity of the problem.  The caller must
    already handle return values of empty strings, so now a missing
    /etc/pcp.conf is treated just as a misformatted one, and results in
    the same "" result.
    
    QA 448 is updated to accept a rc=0 outcome from a non-crashing
    src/getconfig, and the pmgetconfig.3 man page scares less.

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