]> xenbits.xensource.com Git - libvirt.git/commitdiff
Resolve Coverity dead_error_begin
authorJohn Ferlan <jferlan@redhat.com>
Fri, 31 Jan 2014 16:32:28 +0000 (11:32 -0500)
committerJohn Ferlan <jferlan@redhat.com>
Fri, 31 Jan 2014 17:48:01 +0000 (12:48 -0500)
Coverity complains about default: label in libxl_driver.c not be able
to be reached. It's by design for the code and since it's not necessary
in the code nor does it elicit any compiler/make check warnings - just
remove it rather than adding a coverity[dead_error_begin] tag.

While I'm at it, lxc_driver.c and nodeinfo.c have the same design, so I
removed the default labels and the existing coverity tags.

src/libxl/libxl_driver.c
src/lxc/lxc_driver.c
src/nodeinfo.c

index fc0efa2e5a91e765fc829959f8605816cb123fb7..cb3deeccc2cf184cb9d9b7f3fb94ce1b41885513 100644 (file)
@@ -4167,9 +4167,6 @@ libxlDomainGetNumaParameters(virDomainPtr dom,
             nodeset = NULL;
 
             break;
-
-        default:
-            break;
         }
     }
 
index 73a2986710e03f0abe4d8e5a6832052420259b51..138c7061967d8e20d3e104ee330574ec12f5b151 100644 (file)
@@ -950,11 +950,6 @@ lxcDomainGetMemoryParameters(virDomainPtr dom,
                                         VIR_TYPED_PARAM_ULLONG, val) < 0)
                 goto cleanup;
             break;
-
-        /* coverity[dead_error_begin] */
-        default:
-            break;
-            /* should not hit here */
         }
     }
 
@@ -2626,11 +2621,6 @@ lxcDomainGetBlkioParameters(virDomainPtr dom,
                                             param->value.s) < 0)
                     goto cleanup;
                 break;
-
-            /* coverity[dead_error_begin] */
-            default:
-                break;
-                /* should not hit here */
             }
         }
     } else if (flags & VIR_DOMAIN_AFFECT_CONFIG) {
@@ -2818,11 +2808,6 @@ lxcDomainGetBlkioParameters(virDomainPtr dom,
                     goto cleanup;
                 }
                 break;
-
-            /* coverity[dead_error_begin] */
-            default:
-                break;
-                /* should not hit here */
             }
         }
     }
index 6ebfb4b95d64d864cc43f7a26a5635a293e17442..22b139f35cfc69a24551809acb7910951c18aba5 100644 (file)
@@ -1478,10 +1478,6 @@ nodeGetMemoryParameters(virTypedParameterPtr params ATTRIBUTE_UNUSED,
                 return -1;
 
             break;
-
-        /* coverity[dead_error_begin] */
-        default:
-            break;
         }
     }