Since tst_brkm() can break remaining test cases, the code
after this function is redundant.

Signed-off-by: Xing Gu <gux.f...@cn.fujitsu.com>
---
 lib/safe_stdio.c | 1 -
 lib/tst_virt.c   | 1 -
 2 files changed, 2 deletions(-)

diff --git a/lib/safe_stdio.c b/lib/safe_stdio.c
index 3463659..e202386 100644
--- a/lib/safe_stdio.c
+++ b/lib/safe_stdio.c
@@ -32,7 +32,6 @@ FILE *safe_fopen(const char *file, const int lineno, void 
(cleanup_fn)(void),
                tst_brkm(TBROK | TERRNO, cleanup_fn,
                         "%s:%d: fopen(%s,%s) failed",
                         file, lineno, path, mode);
-               return NULL;
        }
 
        return f;
diff --git a/lib/tst_virt.c b/lib/tst_virt.c
index 0141c3d..d1f1e74 100644
--- a/lib/tst_virt.c
+++ b/lib/tst_virt.c
@@ -49,5 +49,4 @@ int tst_is_virt(int virt_type)
                return is_xen();
        }
        tst_brkm(TBROK, NULL, "invalid virt_type flag: %d", virt_type);
-       return 0;
 }
-- 
1.9.0


------------------------------------------------------------------------------
"Accelerate Dev Cycles with Automated Cross-Browser Testing - For FREE
Instantly run your Selenium tests across 300+ browser/OS combos.
Get unparalleled scalability from the best Selenium testing platform available
Simple to use. Nothing to install. Get started now for free."
http://p.sf.net/sfu/SauceLabs
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to