RFR: 8303002: Reject packed structs from linker [v5]
Maurizio Cimadamore
mcimadamore at openjdk.org
Tue Apr 18 22:20:48 UTC 2023
On Tue, 18 Apr 2023 13:45:57 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:
>> This patch adds checks in AbstractLinker to reject packed structs and structs with excess padding (e.g. unnamed bitfields), since both of those are currently not supported, and the ABI/spec seems too vague to base support on.
>
> Jorn Vernee has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains seven additional commits since the last revision:
>
> - fix TestIllegalLink
> - Merge branch 'PR_21' into RejectPacked
> - Merge branch 'PR_21' into RejectPacked
> - add javadoc + fixes for trailing padding
> - fix check for padding. Add check for trailing padding too
> - Reject packed structs and structs with extra padding
> - Check byte order of layouts passed to linker
src/java.base/share/classes/java/lang/foreign/Linker.java line 200:
> 198: * </table></blockquote>
> 199: * <p>
> 200: * Due to limited ABI specification coverage, all the native linker implementations limit the function
`All the native linker implementations support function function descriptors that contain only so-called...`
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13164#discussion_r1170619687
More information about the core-libs-dev
mailing list