Bug 110751 - broken source code
Summary: broken source code
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: ghostscript
Version: 1
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Tim Waugh
QA Contact: Mike McLean
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2003-11-24 13:58 UTC by d.binderman
Modified: 2007-11-30 22:10 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2003-12-06 12:47:56 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description d.binderman 2003-11-24 13:58:30 UTC
From Bugzilla Helper:
User-Agent: Mozilla/4.0 (compatible; MSIE 6.0; Windows 98)

Description of problem:

I just tried to compile package ghostscript-7_07-11 from Fedora.

The compiler said

./src/sdcparam.c(161): remark #592: variable "quant_tables" is used 
before its value is set

The source code is

    gs_param_array quant_tables;

	/* more code */

    num_in_tables = quant_tables.size;

Seems like a pretty obvious bug to me.



Version-Release number of selected component (if applicable):
ghostscript-7_07-11 

How reproducible:
Always

Steps to Reproduce:
1. by inspection
2.
3.
    

Additional info:

Comment 1 Tim Waugh 2003-12-05 16:08:07 UTC
It's very far from clear what the fix is, and the comment immediately
above that line indicates that someone else found the problem and is
also not sure what to do.  Since it's known upstream, I'm closing this.

Comment 2 d.binderman 2003-12-06 09:26:26 UTC
>It's very far from clear what the fix is, 

Fair enough - not all bugs are easy to fix.

>someone else found the problem and is also not sure what to do

fair enough - two people don't know how to fix it.

>Since it's known upstream, I'm closing this

I am unhappy with this. I think the bug needs fixing, independent
of whether two people don't know what the fix is.

Does it only get fixed if the fix is easy ?



Comment 3 Tim Waugh 2003-12-06 12:47:56 UTC
It's not even clear that this is a code path that gets executed.  If
it had been causing some outrageous symptom it might be worth spending
some time working on.  Best worked on by those who know the code best,
upstream.


Note You need to log in before you can comment on or make changes to this bug.