8307850: update for deprecated sprintf for jdk.jdi

Reviewed-by: cjplummer
This commit is contained in:
Xue-Lei Andrew Fan 2023-05-11 18:07:54 +00:00
parent 7fcb0fdcd4
commit cbecf422df
2 changed files with 15 additions and 15 deletions

View File

@ -1,5 +1,5 @@
/*
* Copyright (c) 1999, 2022, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 1999, 2023, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@ -64,9 +64,9 @@ throwShmemException(JNIEnv *env, char *message, jint errorCode)
char buf[255];
if (shmemBase_getlasterror(msg, sizeof(msg)) == SYS_OK) {
sprintf(buf, "%s: %s\n", message, msg);
snprintf(buf, sizeof(buf), "%s: %s\n", message, msg);
} else {
sprintf(buf, "%s, error code = %d", message, errorCode);
snprintf(buf, sizeof(buf), "%s, error code = %d", message, errorCode);
}
throwException(env, "java/io/IOException", buf);
}

View File

@ -241,7 +241,7 @@ createWithGeneratedName(char *prefix, char *nameBuffer, CreateFunc func, void *a
strcpy(nameBuffer, prefix);
if (i > 0) {
char buf[10];
sprintf(buf, ".%d", i+1);
snprintf(buf, sizeof(buf), ".%d", i+1);
strcat(nameBuffer, buf);
}
error = func(nameBuffer, arg);
@ -430,14 +430,14 @@ createStream(char *name, Stream *stream)
jint error;
char objectName[MAX_IPC_NAME];
sprintf(objectName, "%s.mutex", name);
snprintf(objectName, sizeof(objectName), "%s.mutex", name);
error = createWithGeneratedName(objectName, stream->shared->mutexName,
createMutex, &stream->mutex);
if (error != SYS_OK) {
return error;
}
sprintf(objectName, "%s.hasData", name);
snprintf(objectName, sizeof(objectName), "%s.hasData", name);
error = createWithGeneratedName(objectName, stream->shared->hasDataEventName,
createEvent, &stream->hasData);
if (error != SYS_OK) {
@ -445,7 +445,7 @@ createStream(char *name, Stream *stream)
return error;
}
sprintf(objectName, "%s.hasSpace", name);
snprintf(objectName, sizeof(objectName), "%s.hasSpace", name);
error = createWithGeneratedName(objectName, stream->shared->hasSpaceEventName,
createEvent, &stream->hasSpace);
if (error != SYS_OK) {
@ -568,7 +568,7 @@ openConnection(SharedMemoryTransport *transport, jlong otherPID,
return SYS_NOMEM;
}
sprintf(connection->name, "%s.%" PRId64, transport->name, sysProcessGetID());
snprintf(connection->name, sizeof(connection->name), "%s.%" PRId64, transport->name, sysProcessGetID());
error = sysSharedMemOpen(connection->name, &connection->sharedMemory,
&connection->shared);
if (error != SYS_OK) {
@ -636,7 +636,7 @@ createConnection(SharedMemoryTransport *transport, jlong otherPID,
return SYS_NOMEM;
}
sprintf(connection->name, "%s.%" PRId64, transport->name, otherPID);
snprintf(connection->name, sizeof(connection->name), "%s.%" PRId64, transport->name, otherPID);
error = sysSharedMemCreate(connection->name, sizeof(SharedMemory),
&connection->sharedMemory, &connection->shared);
if (error != SYS_OK) {
@ -735,7 +735,7 @@ openTransport(const char *address, SharedMemoryTransport **transportPtr)
if (strlen(address) >= MAX_IPC_PREFIX) {
char buf[128];
sprintf(buf, "Error: address strings longer than %d characters are invalid\n", MAX_IPC_PREFIX);
snprintf(buf, sizeof(buf), "Error: address strings longer than %d characters are invalid\n", MAX_IPC_PREFIX);
setLastErrorMsg(buf);
closeTransport(transport);
return SYS_ERR;
@ -799,7 +799,7 @@ createTransport(const char *address, SharedMemoryTransport **transportPtr)
} else {
if (strlen(address) >= MAX_IPC_PREFIX) {
char buf[128];
sprintf(buf, "Error: address strings longer than %d characters are invalid\n", MAX_IPC_PREFIX);
snprintf(buf, sizeof(buf), "Error: address strings longer than %d characters are invalid\n", MAX_IPC_PREFIX);
setLastErrorMsg(buf);
closeTransport(transport);
return SYS_ERR;
@ -817,7 +817,7 @@ createTransport(const char *address, SharedMemoryTransport **transportPtr)
memset(transport->shared, 0, sizeof(SharedListener));
transport->shared->acceptingPID = sysProcessGetID();
sprintf(objectName, "%s.mutex", transport->name);
snprintf(objectName, sizeof(objectName), "%s.mutex", transport->name);
error = createWithGeneratedName(objectName, transport->shared->mutexName,
createMutex, &transport->mutex);
if (error != SYS_OK) {
@ -825,7 +825,7 @@ createTransport(const char *address, SharedMemoryTransport **transportPtr)
return error;
}
sprintf(objectName, "%s.accept", transport->name);
snprintf(objectName, sizeof(objectName), "%s.accept", transport->name);
error = createWithGeneratedName(objectName, transport->shared->acceptEventName,
createEvent, &transport->acceptEvent);
if (error != SYS_OK) {
@ -833,7 +833,7 @@ createTransport(const char *address, SharedMemoryTransport **transportPtr)
return error;
}
sprintf(objectName, "%s.attach", transport->name);
snprintf(objectName, sizeof(objectName), "%s.attach", transport->name);
error = createWithGeneratedName(objectName, transport->shared->attachEventName,
createEvent, &transport->attachEvent);
if (error != SYS_OK) {
@ -1260,7 +1260,7 @@ exitTransportWithError(char *message, char *fileName,
jint error;
char buffer[500];
sprintf(buffer, "Shared Memory Transport \"%s\", line %d: %s\n",
snprintf(buffer, sizeof(buffer), "Shared Memory Transport \"%s\", line %d: %s\n",
fileName, lineNumber, message);
error = (*jvm)->GetEnv(jvm, (void **)&env, JNI_VERSION_1_2);
if (error != JNI_OK) {