diff options
| author | Panu Matilainen <pmatilai@redhat.com> | 2009-09-07 15:04:01 +0300 |
|---|---|---|
| committer | Panu Matilainen <pmatilai@redhat.com> | 2009-09-07 15:04:01 +0300 |
| commit | 2bdd1a5f5c4b23612a7d87992597b1cd39db86c5 (patch) | |
| tree | 03f73517c610768039bf1dc922d837892a2529c8 /build | |
| parent | d510dfa538ed04b27bd3dc2fa75596f15bd4bee7 (diff) | |
| download | rpm-2bdd1a5f5c4b23612a7d87992597b1cd39db86c5.tar.gz rpm-2bdd1a5f5c4b23612a7d87992597b1cd39db86c5.tar.bz2 rpm-2bdd1a5f5c4b23612a7d87992597b1cd39db86c5.zip | |
Avoid theoretical NULL pointer dereference
- if sbp ends up not set it's an "internal error" really, dont bother
with error message
Diffstat (limited to 'build')
| -rw-r--r-- | build/parseBuildInstallClean.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/build/parseBuildInstallClean.c b/build/parseBuildInstallClean.c index 6f7421e69..d81bd0727 100644 --- 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) { |