Variable 'err' is initially set to 1, so changing its value with '|=' won't set it to 0 even if the operation is successful. Signed-off-by: Maciej Naruszewicz <maciej.naruszewicz@xxxxxxxxx> --- Detail.c | 1 + 1 file changed, 1 insertion(+) diff --git a/Detail.c b/Detail.c index 57faf3c..96d1ff6 100644 --- a/Detail.c +++ b/Detail.c @@ -640,6 +640,7 @@ int Detail_Platform(struct superswitch *ss, int scan, int verbose, int export, c if (!scan) return err; + err = 0; for (i = 0; superlist[i]; i++) { struct superswitch *meta = superlist[i]; -- To unsubscribe from this list: send the line "unsubscribe linux-raid" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html