bug in nsTextFormater.cpp

VERIFIED FIXED in M16

Status

()

Core
XPCOM
P3
normal
VERIFIED FIXED
18 years ago
18 years ago

People

(Reporter: Frank Tang, Assigned: Frank Tang)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Assignee)

Description

18 years ago
Subject: 
        This code must be broken
   Date: 
        Thu, 6 Apr 2000 14:39:26 -0400
   From: 
        "Jim Nance" <jnance@nortelnetworks.com>
     To: 
        mozilla-builds@mozilla.org
    CC: 
        ftang@netscape.com




Hello All,
    I found some code that I think is broken.  Can someone confirm this?
The lxr link is:

    http://lxr.mozilla.org/seamonkey/source/xpcom/ds/nsTextFormater.cpp#361

We are doing stuff with the fout anf fout2 arrays, and they have never been
initialized AFAICT.  Am I right about this?

Thanks,

Jim

Subject: 
             Re: This code must be broken
        Date: 
             Thu, 06 Apr 2000 13:33:04 -0700
       From: 
             ftang@netscape.com (Yung-Fong Tang)
 Organization: 
             Netscape
         To: 
             Jim Nance <jnance@nortelnetworks.com>
         CC: 
             mozilla-builds@mozilla.org
  References: 
             1




Good catch, this file is copy from  mozilla/nsprpub/pr/src/io/prprf.c 
Howerver, it seems we miss one line there. 

344     sprintf(fout, fin, d); 

I will add it.
(Assignee)

Updated

18 years ago
Status: NEW → ASSIGNED
Target Milestone: --- → M16
(Assignee)

Comment 1

18 years ago
fix and check in.
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED

Comment 2

18 years ago
- Per last comments, age of bug, and no reopen - Marking Verified/Fixed.  Please 
reopen if still a problem. 
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.