[netcf-devel] [PATCH 2/4] ifup/ifdown: make sure errors from run1 are propagated

David Lutterkort lutter at redhat.com
Tue Sep 15 17:54:27 UTC 2009


---
 src/drv_initscripts.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/src/drv_initscripts.c b/src/drv_initscripts.c
index e17d834..e928962 100644
--- a/src/drv_initscripts.c
+++ b/src/drv_initscripts.c
@@ -978,6 +978,7 @@ int drv_if_up(struct netcf_if *nif) {
         }
     }
     run1(ncf, ifup, nif->name);
+    ERR_BAIL(ncf);
     result = 0;
  error:
     free_matches(nslaves, &slaves);
@@ -992,6 +993,7 @@ int drv_if_down(struct netcf_if *nif) {
     int result = -1;
 
     run1(ncf, ifdown, nif->name);
+    ERR_BAIL(ncf);
     if (is_bridge(ncf, nif->name)) {
         /* Bring up bridge slaves after the bridge */
         nslaves = bridge_slaves(ncf, nif->name, &slaves);
-- 
1.6.2.5



More information about the netcf-devel mailing list