Bug 661881

Summary: clufindhostname -i returns random value
Product: Red Hat Enterprise Linux 6 Reporter: Lon Hohberger <lhh>
Component: rgmanagerAssignee: Lon Hohberger <lhh>
Status: CLOSED ERRATA QA Contact: Cluster QE <mspqa-list>
Severity: medium Docs Contact:
Priority: low    
Version: 6.0CC: cluster-maint, cmarthal, djansa, dyn_ares, edamato, joshua
Target Milestone: rc   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: rgmanager-3.0.12-11.el6 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: 592613 Environment:
Last Closed: 2011-05-19 14:18:33 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 592613    
Bug Blocks:    

Description Lon Hohberger 2010-12-09 21:06:53 UTC
+++ This bug was initially created as a clone of Bug #592613 +++

Description of problem:
clufindhostname -i myhost.example.com returns random value.

Version-Release number of selected component (if applicable):
Tested with 2.0.56-6 (latest) 

How reproducible:
Every time.. 


Steps to Reproduce:
1. clufindhostname -i myhost.example.com 
2. most likely it will return a funny value and make nfsclient check in the cluster go south.

Problem is that:
inet_pton:
0 is returned if src does not contain  a  character  string  representing  a  valid  network address. A positive value is returned if the network address was successfully converted.

From  clufindhostname source code:

if (inet_pton(family, optarg, ptr) < 0) {
        perror("inet_pton");
        exit(2);
}

The check should be:

if (inet_pton(family, optarg, ptr) <= 0) {
        perror("inet_pton");
        exit(2);
}

Cheers D

--- Additional comment from lhh on 2010-12-09 16:06:20 EST ---

Patch pushed upstream:

http://git.fedorahosted.org/git?p=cluster.git;a=commit;h=b057008da82f0caeb539e33575eb1f0a07da192c

Comment 4 Lon Hohberger 2011-03-18 15:10:32 UTC
*** Bug 670630 has been marked as a duplicate of this bug. ***

Comment 5 joshua 2011-03-18 15:31:21 UTC
Lon, do you have an ETA on when the fixed rgmanager will be pushed out?  clufindhostname is pretty darn important to the basic operation of cluster services

Comment 6 Corey Marthaler 2011-04-14 22:44:40 UTC
Fix verified in rgmanager-3.0.12-11.el6.x86_64.

[root@taft-04 ~]# clufindhostname -n taft-01
taft-01.lab.msp.redhat.com
[root@taft-04 ~]# clufindhostname -n taft-02
taft-02.lab.msp.redhat.com

[root@taft-04 ~]# clufindhostname -i 10.15.89.67
taft-01.lab.msp.redhat.com
[root@taft-04 ~]# clufindhostname -i 10.15.89.68
taft-02.lab.msp.redhat.com

Comment 7 errata-xmlrpc 2011-05-19 14:18:33 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHBA-2011-0750.html