RE: Bug fixes

Hi Jens

You may well have already noticed this, but in your proposed change to
HTUserProfile_delete() below, all you're doing is assigning a new value to
a pass-by-value parameter, just before the routine returns - effectively a
no-op !!
The only place it is called from is HTLibTerminate() though, and Patch 4
provides the required assign-to-null operation after this call returns.

For Patch 9, I can confirm that the code I patched was exactly as in your
post below, but unfortunately I didn't keep too many notes. This was back
in January, but what I vaguely remember is that Purify reported an FMR
(Freed Memory Read), and when I went through the code manually, with the
aid of some trace statements, I was able to see why.
Looking at the code visually, I must admit that I don't see any functional
difference between my patch and the old code. I've also just tried a quick
test run using the old code, without finding the problem.
??? Pass - I seem to have gotten confused somewhere.


By the way, I also have one more request.
The HTTP_setBodyWriteDelay() routine in HTTP.c currently checks the
parameters, and runs some checks to make sure the delay is at least 20
milliseconds. Is there a reason for this - why not just accept whatever
values the progammer requests (including zero ?)

Regards
Yusef






Jens Meggers <jens.meggers@firepad.com>@w3.org on 28/04/2001 02:49:46

Sent by:  www-lib-request@w3.org


To:   "'Yusef_Badri@tertio.com'" <Yusef_Badri@tertio.com>, www-lib@w3.org
cc:   yb@greybox.demon.co.uk
Subject:  RE: Bug fixes


Yusef,

thanks for providing the bug fixes. I checked all fixes. I have the
following comments:

Patch 1: OK
Patch 2: OK
Patch 3:
This is a good improvement. It seems to me that we also should set the user
profgile to NULL if it is once deleted.
Thus, I also added the up = NULL in HTUserProfile_delete(). Seems to me
that
nobody paid attention to this because they never reenter the lib.

PUBLIC BOOL HTUserProfile_delete (HTUserProfile * up)
{
    if (up) {
     HT_FREE(up->user);
     HT_FREE(up->fqdn);
     HT_FREE(up->email);
     HT_FREE(up->news);
     HT_FREE(up->tmp);
     HT_FREE(up);
     up = NULL;
     return YES;
    }
    return NO;
}


Patch 4:
Ok, I have done it already in 3. Seems to be a little bit more secure as
HTUserProfile_delete() cannot free two times.


Patch 5
Ok.

Patch 6
OK

Patch 7
Ok

Patch 8
ok

Patch 9

Your changes are looking ok, but I do not see the mem free error in the old
implmentation. I currently have the following code:

    if (cookie_holder) {

          HTList * cur = cookie_holder;
          HTCookieHolder * pres = NULL;
          while ((pres = (HTCookieHolder *) HTList_nextObject(cur))) {

              HTCookieHolder_delete(pres);

          }

          HTList_delete(cookie_holder);
          cookie_holder = NULL;
          return YES;
    }

What is wrong with that?


Patch 10

ok. I also would perefer not to change the wwwstr.c function.


Patch 11

ok.

Patch 12

ok.



Thanks again for the input.


jens

Received on Monday, 30 April 2001 08:55:55 UTC