-
Notifications
You must be signed in to change notification settings - Fork 722
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Build error: "storing the address of local variable" in runtime/rastrace/trcmain.c #20171
Labels
Comments
Another build error with GCC 13.2.0: #20170 |
knn-k
added a commit
to knn-k/openj9
that referenced
this issue
Sep 17, 2024
This commit stops storing the address of a local variable, to fix a build error with runtime/rastrace/trcmain.c. Fixes: eclipse-openj9#20171 Signed-off-by: KONNO Kazuhiro <[email protected]>
I opened a draft PR: #20172 |
2 tasks
knn-k
added a commit
to knn-k/openj9
that referenced
this issue
Sep 26, 2024
This commit fixes a build error with runtime/rastrace/trcmain.c. Fixes: eclipse-openj9#20171 Signed-off-by: KONNO Kazuhiro <[email protected]>
rmnattas
pushed a commit
to rmnattas/openj9
that referenced
this issue
Oct 18, 2024
This commit fixes a build error with runtime/rastrace/trcmain.c. Fixes: eclipse-openj9#20171 Signed-off-by: KONNO Kazuhiro <[email protected]>
rmnattas
pushed a commit
to rmnattas/openj9
that referenced
this issue
Nov 1, 2024
This commit fixes a build error with runtime/rastrace/trcmain.c. Fixes: eclipse-openj9#20171 Signed-off-by: KONNO Kazuhiro <[email protected]>
zl-wang
pushed a commit
to zl-wang/openj9
that referenced
this issue
Nov 11, 2024
This commit fixes a build error with runtime/rastrace/trcmain.c. Fixes: eclipse-openj9#20171 Signed-off-by: KONNO Kazuhiro <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Building OpenJ9 Java 17 with GCC 13.2.0 on Ubuntu 24.04.1 generates the following error:
openj9/runtime/rastrace/trcmain.c
Line 1045 in 0985ff3
tempThr
is a local variable ininitializeTrace()
.thr
is the first argument forinitializeTrace()
. The address oftempThr
is used byJ9VMDllMain()
in trcengine.c, the caller ofinitializeTrace()
.openj9/runtime/rastrace/trcengine.c
Line 363 in 0985ff3
The memory region for
tempThr
in the stack can be overwritten by other functions.I think
J9VMDllMain()
should prepare the memory region forUtThreadData
, asJ9VMDllMain()
seems to be the only caller ofinitializeTrace()
.The text was updated successfully, but these errors were encountered: