The while loop above the dead code already contains an error check, so this additional one will never be triggered. Found by Coverity. --- osinfo/osinfo_loader.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/osinfo/osinfo_loader.c b/osinfo/osinfo_loader.c index 46bc633..3168e9b 100644 --- a/osinfo/osinfo_loader.c +++ b/osinfo/osinfo_loader.c @@ -2118,9 +2118,6 @@ static void osinfo_loader_process_list(OsinfoLoader *loader, tmp++; } - if (lerr) - goto cleanup; - /* Phase 2: load data from non-native locations, filtering based * on overrides from native locations */ tmp = dirs; -- 2.7.4 _______________________________________________ Libosinfo mailing list Libosinfo@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libosinfo