I'm no expert, but resources/OCF/IPaddr.in looks broken to me.

The first is this conditional which looks new:

        if [ ! -z "$OCF_RESKEY_netmask" ]; then
                NICINFO=`$FINDIF "$BASEIP/$NETMASK/$NIC/$BRDCAST"`
        else
                NICINFO=`$FINDIF "$BASEIP"`
        fi

This makes no sense as the next line is:
        NICINFO=`$FINDIF "$BASEIP/$NETMASK/$NIC/$BRDCAST"`

I also believe these lines require a -C argument so that when the netmask is parsed and fed back into $FINDIF later, $FINDIF doesn't barf.

The reason $FINDIF doesn't barf now is that the calculated netmask is never used. This too is fixed in the attached patch.

I have also included some code to make the parsing of $FINDIF output work on Darwin.
Possibly its broken on other platforms too.

Permission to apply?


Attachment: ipaddr.diff
Description: Binary data


--
Andrew Beekhof

"Would the last person to leave please turn out the enlightenment?" - TISM

_______________________________________________________
Linux-HA-Dev: [email protected]
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/

Reply via email to