diff options
| author | Josephine Pfeiffer <hi@josie.lol> | 2025-10-01 19:40:46 +0200 |
|---|---|---|
| committer | Janosch Frank <frankja@linux.ibm.com> | 2025-10-20 12:43:18 +0000 |
| commit | f5a6fa189ad2e73d25e31de3cc7f0bf81907c986 (patch) | |
| tree | fb119b498a637e426d5ada9fdb8e4426a7fcf6a1 | |
| parent | 7d5136ed1b218f7d68e15ff7a6d000a7ff3cce0f (diff) | |
KVM: s390: Replace sprintf with snprintf for buffer safety
Replace sprintf() with snprintf() when formatting debug names to prevent
potential buffer overflow. The debug_name buffer is 16 bytes, and while
unlikely to overflow with current PIDs, using snprintf() provides proper
bounds checking.
Signed-off-by: Josephine Pfeiffer <hi@josie.lol>
[frankja@linux.ibm.com: Fixed subject prefix]
Signed-off-by: Janosch Frank <frankja@linux.ibm.com>
| -rw-r--r-- | arch/s390/kvm/kvm-s390.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c index 677aa5c7d226..70ebc54b1bb1 100644 --- a/arch/s390/kvm/kvm-s390.c +++ b/arch/s390/kvm/kvm-s390.c @@ -3348,7 +3348,7 @@ int kvm_arch_init_vm(struct kvm *kvm, unsigned long type) if (!kvm->arch.sca) goto out_err; - sprintf(debug_name, "kvm-%u", current->pid); + snprintf(debug_name, sizeof(debug_name), "kvm-%u", current->pid); kvm->arch.dbf = debug_register(debug_name, 32, 1, 7 * sizeof(long)); if (!kvm->arch.dbf) |