Avoid theoretical NULL pointer dereference

- if sbp ends up not set it's an "internal error" really, dont bother
  with error message

diff --git a/build/parseBuildInstallClean.c b/build/parseBuildInstallClean.c
index 6f7421e..d81bd07 100644 (file)
--- a/build/parseBuildInstallClean.c
+++ b/build/parseBuildInstallClean.c
@@ -27,6 +27,8 @@ int parseBuildInstallClean(rpmSpec spec, rpmParseState parsePart)
     } else if (parsePart == PART_CLEAN) {
        sbp = &(spec->clean);
        name = "%clean";
+    } else {
+       goto exit; /* programmer error */
     }
     
     if (*sbp != NULL) {