8335918: update for deprecated sprintf for jvmti

gdams · gdams · commit e43402d79f76 · 2024-07-09T07:47:05.000Z
Reviewed-by: sgehwolf
diff --git a/test/hotspot/jtreg/vmTestbase/nsk/jvmti/ThreadEnd/threadend001/threadend001.cpp b/test/hotspot/jtreg/vmTestbase/nsk/jvmti/ThreadEnd/threadend001/threadend001.cpp
@@ -57,7 +57,7 @@ void JNICALL ThreadEnd(jvmtiEnv *jvmti_env, JNIEnv *env, jthread thread) {
     }
     if (inf.name != NULL && strstr(inf.name, prefix) == inf.name) {
         eventsCount++;
-        sprintf(name, "%s%d", prefix, eventsCount);
+        snprintf(name, sizeof(name), "%s%d", prefix, eventsCount);
         if (inf.name == NULL || strcmp(name, inf.name) != 0) {
             printf("(#%d) wrong thread name: \"%s\"",
                    eventsCount, inf.name);
diff --git a/test/hotspot/jtreg/vmTestbase/nsk/jvmti/ThreadStart/threadstart001/threadstart001.cpp b/test/hotspot/jtreg/vmTestbase/nsk/jvmti/ThreadStart/threadstart001/threadstart001.cpp
@@ -56,7 +56,7 @@ void JNICALL ThreadStart(jvmtiEnv *jvmti_env, JNIEnv *env, jthread thread) {
         printf(">>> %s\n", inf.name);
     }
     if (inf.name != NULL && strstr(inf.name, prefix) == inf.name) {
-        sprintf(name, "%s%d", prefix, eventsCount);
+        snprintf(name, sizeof(name), "%s%d", prefix, eventsCount);
         if (strcmp(name, inf.name) != 0) {
             printf("(#%d) wrong thread name: \"%s\"",
                    eventsCount, inf.name);

Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@ void JNICALL ThreadEnd(jvmtiEnv jvmti_env, JNIEnv env, jthread thread) {`
`57`	`57`	`}`
`58`	`58`	`if (inf.name != NULL && strstr(inf.name, prefix) == inf.name) {`
`59`	`59`	`eventsCount++;`
`60`		`- sprintf(name, "%s%d", prefix, eventsCount);`
	`60`	`+ snprintf(name, sizeof(name), "%s%d", prefix, eventsCount);`
`61`	`61`	`if (inf.name == NULL \|\| strcmp(name, inf.name) != 0) {`
`62`	`62`	`printf("(#%d) wrong thread name: \"%s\"",`
`63`	`63`	`eventsCount, inf.name);`
Original file line number	Diff line number	Diff line change
`@@ -56,7 +56,7 @@ void JNICALL ThreadStart(jvmtiEnv jvmti_env, JNIEnv env, jthread thread) {`
`56`	`56`	`printf(">>> %s\n", inf.name);`
`57`	`57`	`}`
`58`	`58`	`if (inf.name != NULL && strstr(inf.name, prefix) == inf.name) {`
`59`		`- sprintf(name, "%s%d", prefix, eventsCount);`
	`59`	`+ snprintf(name, sizeof(name), "%s%d", prefix, eventsCount);`
`60`	`60`	`if (strcmp(name, inf.name) != 0) {`
`61`	`61`	`printf("(#%d) wrong thread name: \"%s\"",`
`62`	`62`	`eventsCount, inf.name);`