On Tue, 2 Apr 2024 14:59:33 GMT, Sean Coffey wrote:
>> Calling extra logging calls during initialization of Logger libraries can
>> cause recursion leading to StackOverflowError
>> This patch adds logic to the EventHelper class to detect recursion and
>> prevent it.
>
> Sean Coffey has updated
On Tue, 2 Apr 2024 14:59:33 GMT, Sean Coffey wrote:
>> Calling extra logging calls during initialization of Logger libraries can
>> cause recursion leading to StackOverflowError
>> This patch adds logic to the EventHelper class to detect recursion and
>> prevent it.
>
> Sean Coffey has updated
On Tue, 2 Apr 2024 13:38:00 GMT, Daniel Fuchs wrote:
>> Sean Coffey has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> incorporate testcase feedback from Daniel
>
> test/jdk/jdk/security/logging/RecursiveEventHelper.java line 56:
>
>> 54:
On Tue, 2 Apr 2024 15:01:15 GMT, Sean Coffey wrote:
> I guess the question is what sort of Logger and LoggerFinder would you return
> in cases where recursion is detected ? Extra issues might arise from the
> System.getLogger method being public and widely used. Additional issues might
> stem
On Tue, 2 Apr 2024 14:59:33 GMT, Sean Coffey wrote:
>> Calling extra logging calls during initialization of Logger libraries can
>> cause recursion leading to StackOverflowError
>> This patch adds logic to the EventHelper class to detect recursion and
>> prevent it.
>
> Sean Coffey has updated
On Tue, 2 Apr 2024 11:16:55 GMT, Jaikiran Pai wrote:
>> Calling extra logging calls during initialization of Logger libraries can
>> cause recursion leading to StackOverflowError
>> This patch adds logic to the EventHelper class to detect recursion and
>> prevent it.
>
> You are right, turns ou
> Calling extra logging calls during initialization of Logger libraries can
> cause recursion leading to StackOverflowError
> This patch adds logic to the EventHelper class to detect recursion and
> prevent it.
Sean Coffey has updated the pull request incrementally with one additional
commit si
On Thu, 28 Mar 2024 15:55:12 GMT, Sean Coffey wrote:
> Calling extra logging calls during initialization of Logger libraries can
> cause recursion leading to StackOverflowError
> This patch adds logic to the EventHelper class to detect recursion and
> prevent it.
The code changes LGTM. Some co
On Thu, 28 Mar 2024 15:55:12 GMT, Sean Coffey wrote:
> Calling extra logging calls during initialization of Logger libraries can
> cause recursion leading to StackOverflowError
> This patch adds logic to the EventHelper class to detect recursion and
> prevent it.
You are right, turns out the j
On Thu, 28 Mar 2024 15:55:12 GMT, Sean Coffey wrote:
> Calling extra logging calls during initialization of Logger libraries can
> cause recursion leading to StackOverflowError
> This patch adds logic to the EventHelper class to detect recursion and
> prevent it.
Thanks for taking a look at th
On Thu, 28 Mar 2024 15:55:12 GMT, Sean Coffey wrote:
> Calling extra logging calls during initialization of Logger libraries can
> cause recursion leading to StackOverflowError
> This patch adds logic to the EventHelper class to detect recursion and
> prevent it.
Hello Sean, from what I unders
Calling extra logging calls during initialization of Logger libraries can cause
recursion leading to StackOverflowError
This patch adds logic to the EventHelper class to detect recursion and prevent
it.
-
Commit messages:
- 8329013
Changes: https://git.openjdk.org/jdk/pull/18534/f
12 matches
Mail list logo
