Rémy,
On 5/15/24 09:12, Rémy Maucherat wrote:
On Tue, May 14, 2024 at 11:15 PM Christopher Schultz
<ch...@christopherschultz.net> wrote:
All,
I'd like to basically globally-search-and-replace ERR_error_string for
ERR_error_string_n and use a #define constant for both the
initialization of all
char err[256];
and similar strings and use that same constant for all calls to
ERR_error_string_n..
Any objections?
There should really be no effective change, except:
1. We can raise that error message length constant and have it affect
the whole library if we choose.
2. We will be using a length-aware string-manipulation call which is
better than using one that assumes that the buffer is at least 256 bytes
long.
+1
This gives me something to do since I thought this was 128 (this
probably came from the tomcat-native code somewhere initially), so I
have a problem with the FFM code which I will fix at the same time. It
seems 128 is already enough in practice.
I already have a patch ready to go. I was just waiting on some feedback
before pushing.
-chris
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org