DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 662CF3858D35
Date: Mon, 22 Nov 2021 11:34:12 +0100
To: cygwin@cygwin.com
Subject: Re: possible snprintf() regression in 3.3.2
Message-ID: <YZtyJDXdrMzVR2lJ@calimero.vinschen.de>
Mail-Followup-To: cygwin@cygwin.com
References: <20211117003718.GF10332@venus.tony.develop-help.com>
 <20211117182108.b38599f5e13071bf269a0d48@nifty.ne.jp>
 <YZT1S8wDnaBuYf5u@calimero.vinschen.de>
 <20211118000649.GG10332@venus.tony.develop-help.com>
 <20211118203538.a049809d57731fe375801c15@nifty.ne.jp>
 <YZZSzPhbqF6cQAiu@calimero.vinschen.de>
 <fa647f83-c5ed-6dea-fcba-c8b799abe12f@gmail.com>
 <YZZi3rI5msk6ksAV@calimero.vinschen.de>
 <7545bb24-43de-cd7d-0764-55c85f1af957@gmx.com>
 <20211121001613.GH10332@venus.tony.develop-help.com>
MIME-Version: 1.0
Content-Disposition: inline
In-Reply-To: <20211121001613.GH10332@venus.tony.develop-help.com>
Precedence: list
From: Corinna Vinschen via Cygwin <cygwin@cygwin.com>
Reply-To: cygwin@cygwin.com
Cc: Corinna Vinschen <corinna-cygwin@cygwin.com>
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: cygwin-bounces+archive-cygwin=delorie.com@cygwin.com
Sender: "Cygwin" <cygwin-bounces+archive-cygwin=delorie.com@cygwin.com>

On Nov 21 11:16, Tony Cook wrote:
> On Thu, Nov 18, 2021 at 09:08:40PM +0000, Sam Edge via Cygwin wrote:
> > I use newlib on embedded with threading libs that have predetermined
> > fixed thread stack sizes. While we tend to have more RAM than in former
> > times we also have multiple thread stacks. Use of alloca() or variable
> > length automatic arrays makes me wince especially in code I might not be
> > able to avoid calling which is often the case with XXXprintf() in
> > third-party libraries' debug output. I'd usually rather take the
> > performance hit from using heap instead of having to make all my stacks
> > bigger.
> 
> A simple option would be to use an small auto fixed buffer for most
> conversions, but use malloc() for %f formats for numbers greater in
> magnitude than some limit, though it would also need to be adjusted
> for the precision (ndigits here), since they take extra space.
> 
> This would avoid using the optional-to-implement VLA feature too.

Good idea.  I guess I create a simple fix doing just that.


Thanks,
Corinna

-- 
Problem reports:      https://cygwin.com/problems.html
FAQ:                  https://cygwin.com/faq/
Documentation:        https://cygwin.com/docs.html
Unsubscribe info:     https://cygwin.com/ml/#unsubscribe-simple