RFR: JDK-8277029: JMM GetDiagnosticXXXInfo APIs should verify output array sizes
David Holmes
dholmes at openjdk.java.net
Tue Nov 16 02:15:34 UTC 2021
On Tue, 16 Nov 2021 01:54:28 GMT, Serguei Spitsyn <sspitsyn at openjdk.org> wrote:
>> Well, it warns in case of programming errors. I agree, that programming error is very unlikely since the caller has all the information he needs. And he could pass in the wrong count.
>>
>> Idk. Mostly I changed this interface to follow the established pattern of always passing in an explicit output buffer size. And to keep it symmetric to``jmm_GetDiagnosticCommandArgumentsInfo` and `jmm_GetGCExtAttributeInfo`. If you object, I'll remove this part.
>
> Hi Thomas,
>
> I kind of agree with David.
> The symmetry does not help in this case as it creates a bit of confusion. :)
>
> Thanks,
> Serguei
Thomas, please drop this part. Thanks.
-------------
PR: https://git.openjdk.java.net/jdk/pull/6363
More information about the serviceability-dev
mailing list