Access violation when "viewing cookies"

VERIFIED FIXED in M4

Status

()

Core
Networking
P3
normal
VERIFIED FIXED
19 years ago
18 years ago

People

(Reporter: jlupa, Assigned: Stephen P. Morse)

Tracking

Trunk
x86
Windows 95
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

19 years ago
in file ?/network/protocol/http/mkaccess.c
in function XP_MakeRawHTMLDialog
(I have a current CVS file) but in mine, line 3915, specificly:
-->fread(readBuf, sizeof(char), fileLength, f);

f can be null generating a access violation, bringing the program down. Simple
error checking can prevent this:

f = fopen(COOKIE_FILE, "r");
if (f)
   fread(readBuf, sizeof(char), fileLength, f);
else /* Things are going to hell in a handbasket.  Dump */
{
   XP_FREE(readBuf);
   return NULL;
}

PLEASE: This is my first "bug report".  I am now trying to run mozilla's browser
as my web browser, and every time it crashes on my, I fix it. If you don't want
these kinds of bugs written up, please email me (jlupa@csisolutions.com), and I
will stop. If you find this kind of bug report useful, email me and I will keep
them coming. =)  Also if you have any tips about figuring out file versions,
etc, I am new to CVS and could use pointers which would make my bug reports
better.

Updated

19 years ago
Assignee: gagan → morse
(Assignee)

Updated

19 years ago
Status: NEW → ASSIGNED
Target Milestone: M4
(Assignee)

Comment 1

19 years ago
I find your kind of bug reporting not only useful but a terrific time saver.
Not only do you report the bugs but you give the fixes as well.  Please don't
stop -- we need more people like you.  Thanks.
(Assignee)

Updated

19 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED
(Assignee)

Comment 2

19 years ago
Made the indicated change to mkaccess.c and checked it in.

Updated

19 years ago
Status: RESOLVED → VERIFIED

Comment 3

19 years ago
since the reporter provided the fix, and morse checked it in, I will mark
verified.

Comment 4

19 years ago
Changing all Networking Library/Browser bugs to Networking-Core component for
Browser.

Occasionally, Bugzilla will burp and cause Verified bugs to reopen when I do
this in a bulk change.  If this happens, I will fix. ;-)

Comment 5

18 years ago
Bulk move of all Networking-Core (to be deleted component) bugs to new
Networking component.
You need to log in before you can comment on or make changes to this bug.