diff options
author | Ken Smith <kensmith@FreeBSD.org> | 2007-12-29 04:56:07 +0000 |
---|---|---|
committer | Ken Smith <kensmith@FreeBSD.org> | 2007-12-29 04:56:07 +0000 |
commit | c0523a3bb58f88507b7216e555bb1958234b80c9 (patch) | |
tree | 20d53864f77ccb70612c29a07dc84e162975232d /usr.sbin | |
parent | 03a3a9133fe6b52753938efda9d816b455e04c7c (diff) | |
download | src-c0523a3bb58f88507b7216e555bb1958234b80c9.tar.gz src-c0523a3bb58f88507b7216e555bb1958234b80c9.zip |
Plug a memory leak. Once any given package (e.g. perl) was installed
we would leak a saved screen for every other package we tried to install
that listed perl as one of its dependencies. When installing things
like gnome and kde that wound up being a LOT of leaked memory.
Insta-MFC request coming so this can be tested as part of 6.3-RC2...
Testing help from: kris
Notes
Notes:
svn path=/head/; revision=174964
Diffstat (limited to 'usr.sbin')
-rw-r--r-- | usr.sbin/sysinstall/index.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/usr.sbin/sysinstall/index.c b/usr.sbin/sysinstall/index.c index 7a9443a73da4..a55b54113dbf 100644 --- a/usr.sbin/sysinstall/index.c +++ b/usr.sbin/sysinstall/index.c @@ -687,7 +687,7 @@ index_extract(Device *dev, PkgNodePtr top, PkgNodePtr who, Boolean depended) int status = DITEM_SUCCESS; PkgNodePtr tmp2; IndexEntryPtr id = who->data; - WINDOW *w = savescr(); + WINDOW *w; /* * Short-circuit the package dependency checks. We're already @@ -702,6 +702,7 @@ index_extract(Device *dev, PkgNodePtr top, PkgNodePtr who, Boolean depended) if (id->installed == 1) return DITEM_SUCCESS; + w = savescr(); if (id && id->deps && strlen(id->deps)) { char t[2048 * 8], *cp, *cp2; @@ -744,6 +745,7 @@ index_extract(Device *dev, PkgNodePtr top, PkgNodePtr who, Boolean depended) dev->volume, id->volume); DEVICE_INIT(mediaDevice); } else { + restorescr(w); return DITEM_FAILURE; } } |