[libvirt] [PATCH 3/6] lxc: Converting full string entries in types only.

Julio Faracco jcfaracco at gmail.com
Tue Feb 26 14:08:59 UTC 2019


Hi John,

I'm seeing clang warnings after applying patches.
Clang is complaining about this line:

> -    if (STREQ(name, "lxc.network.type")) {
> +    if (STREQ(name, "type")) {
>          virDomainDefPtr def = parseData->def;

I think it is irrelevant because there is no way to get a NULL pointer
from case structure.
During my tests, I'm being redirected to "Unhandled network property".
Technically, only lxcNetworkParseDataSuffix is using this method.
Btw, here is a possible fix for that...

--- a/src/lxc/lxc_native.c
+++ b/src/lxc/lxc_native.c
@@ -612,7 +612,7 @@ lxcNetworkParseDataIPs(const char *name,
     if (VIR_ALLOC(ip) < 0)
         return -1;

-    if (STREQ(name, "ipv6"))
+    if (STREQ_NULLABLE(name, "ipv6"))
         family = AF_INET6;

     ipparts = virStringSplit(value->str, "/", 2);

--
Julio Cesar Faracco




More information about the libvir-list mailing list