--- Comment #13 from pinskia at gcc dot gnu dot org 2007-07-30 09:30
---
Fixed as mentioned so closing.
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #12 from tkoenig at gcc dot gnu dot org 2007-07-30 06:11
---
Fixed on trunk. Closing.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32858
--- Comment #11 from tkoenig at gcc dot gnu dot org 2007-07-29 20:01
---
Subject: Bug 32858
Author: tkoenig
Date: Sun Jul 29 20:01:45 2007
New Revision: 127049
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=127049
Log:
2007-07-29 Thomas Koenig <[EMAIL PROTECTED]>
PR
--- Comment #10 from patchapp at dberlin dot org 2007-07-29 18:55 ---
Subject: Bug number PR 32858
A patch for this bug has been added to the patch tracker.
The mailing list url for the patch is
http://gcc.gnu.org/ml/gcc-patches/2007-07/msg02070.html
--
http://gcc.gnu.org/bugzilla/
--- Comment #9 from tkoenig at gcc dot gnu dot org 2007-07-29 16:33 ---
Created an attachment (id=13999)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=13999&action=view)
Patch (current status)
This patch is currently bootstrapping on my machine.
Let's see how it works.
--
tkoe
--- Comment #8 from jb at gcc dot gnu dot org 2007-07-29 15:58 ---
I had a look at your patch, and one thing which looks worrying is the use of
sprintf all over the place. That's just a recipe for buffer overflows,
especially when combined with %s formatting.
I think Tobi's suggestion t
--- Comment #7 from tkoenig at gcc dot gnu dot org 2007-07-29 08:39 ---
I think I understand what's wrong with my patch now: The
stream initialized with init_error_stream was never flushed.
I think I'll go with a naked write() call, which is
a) simpler
b) more robust.
--
http://g
--- Comment #6 from tkoenig at gcc dot gnu dot org 2007-07-28 22:04 ---
There are also a few other issues with the incomplete patch.
vsnprintf can be replaced by __builtin_vsnprintf
on systems where it isn't available. I'll iron that out
when I have the main functionality working.
--
--- Comment #5 from tkoenig at gcc dot gnu dot org 2007-07-28 21:59 ---
Created an attachment (id=13995)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=13995&action=view)
a patch. Unfortunately, it doesn't print out any error messages right now.
--
http://gcc.gnu.org/bugzilla/
--- Comment #4 from burnus at gcc dot gnu dot org 2007-07-25 11:39 ---
Reminder: If implemented, change the PACK library function to print out the
size of LHS/RHS array (cf. PR 30814 & PR 32890).
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32858
--- Comment #3 from jb at gcc dot gnu dot org 2007-07-25 08:46 ---
Related to this, a string library for libgfortran:
http://gcc.gnu.org/ml/fortran/2007-07/msg00463.html
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32858
--- Comment #2 from tkoenig at gcc dot gnu dot org 2007-07-25 05:47 ---
Working on a patch.
I have already gotten rid of st_sprintf.
The nice thing about vsprintf is that it has a gcc builtin, so
we should be able to use it even if the system doesn't have it.
--
tkoenig at gcc dot
--- Comment #1 from jb at gcc dot gnu dot org 2007-07-24 19:06 ---
Confirmed. Also, st_sprintf could be directly replaced by s(n)printf.
--
jb at gcc dot gnu dot org changed:
What|Removed |Added
-
13 matches
Mail list logo
