Relay-Version: version B 2.10 5/3/83; site utzoo.UUCP
Posting-Version: version B 2.10.2 9/17/84 chuqui version 1.7 9/23/84; site nsc.UUCP
Path: utzoo!utcs!lsuc!pesnta!nsc!chuqui
From: chuqui@nsc.UUCP (Chuq Von Rospach)
Newsgroups: net.sources.bugs
Subject: minor savenews bug
Message-ID: <2861@nsc.UUCP>
Date: Mon, 17-Jun-85 12:36:17 EDT
Article-I.D.: nsc.2861
Posted: Mon Jun 17 12:36:17 1985
Date-Received: Tue, 18-Jun-85 08:44:14 EDT
Distribution: net
Organization: Plaidhenge
Lines: 18

It has been pointed out that there is a minor bug in my savenews
news archiver. In the function pop_dir, I accidently switched two
lines and was setting a pointer to NULL before I was freeing the pointer.
On some systems this will cause a segmentation fault, at best you simply
never recover your malloc()ed memory. The two lines should be:

     free(dirstack[lastdir]);
     dirstack[lastdir] = NULL;

instead of the other way around.

chuq
-- 
:From the misfiring synapses of:                  Chuq Von Rospach
{cbosgd,fortune,hplabs,ihnp4,seismo}!nsc!chuqui   nsc!chuqui@decwrl.ARPA

The offices were very nice, and the clients were only raping the land, and
then, of course, there was the money...