[Libvir] [PATCH]Change MAC address to case insensitive

Jim Meyering jim at meyering.net
Tue Feb 26 11:13:43 UTC 2008


"Richard W.M. Jones" <rjones at redhat.com> wrote:
> I'll apply this today (using our STRCASEEQ macros) if no one else
> objects.

Hi,

What do you think about using a MAC-address-specific comparison
function?  I.e., one that is not only case-independent, but that also
knows leading zeros are unnecessary?

i.e., it would admit that these two match:

  00:0A:FF:3A:00:09
  0:a:ff:3a:0:9

That would be a little more user friendly.

In any case, even if we stick with simply case-ignoring, I suggest
we give it a name, like macCompare and use it consistently.
There's at least one other MAC comparison: virsh.c currently uses
xmlStrcasecmp in cmdDetachInterface:

                diff_mac = xmlStrcasecmp(tmp_mac, BAD_CAST mac);




More information about the libvir-list mailing list