HomeClusterLabs Projects

Log: liblrmd: Drop crm_perror() in lrmd_client.c

Description

Log: liblrmd: Drop crm_perror() in lrmd_client.c

Neither of these should have used crm_perror() in the first place,
because errno is not guaranteed to be meaningful. lrmd_send_xml() and
lrmd_send_xml_no_reply() return a legacy Pacemaker return code but
don't set errno explicitly.

The only thing we lose here is writing the message to stderr for CLI
tools. However, CLI tools have verbose options. It doesn't seem worth
passing pcmk__output_t down the call chain... which would be even more
of a pain since it would have to go in the lrmd_t private data somehow.

Other errors in this file already simply use crm_err().

Ref T172

Signed-off-by: Reid Wahl <nrwahl@protonmail.com>

Details

Provenance
nrwahl2Authored on Thu, Aug 7, 5:38 PM
Parents
rPc24424572fe5: Log: libcib: Drop crm_perror() in cib_file.c
Branches
Unknown
Tags
Unknown
Tasks
Restricted Maniphest Task