
16 Aug
2012
16 Aug
'12
4:28 p.m.
On 08/16/2012 02:34 AM, Osier Yang wrote:
- if (virMacAddrParse(macstr,&oldmac) != 0) { + if ((vlan = strchr(fileData, '\n'))) { + char *endptr; + + *vlan++ = 0; /* NULL terminate the mac address */
s/terminate/terminates/
Actually, 'terminate' is correct - this is a comment describing what the assignment is good for (convert arbitrary contents into 0 in order to NUL-terminate the vlan variable), not the vlan format itself (a valid vlan exists when NUL terminates the string). -- Eric Blake eblake@redhat.com +1-919-301-3266 Libvirt virtualization library http://libvirt.org