C 函数中的堆栈粉碎 return

Question

我有一个小程序可以将 12 小时制转换为 24 小时制。

#include <math.h>
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <assert.h>
#include <limits.h>
#include <stdbool.h>

int get_tokens(char* buf, char *fields[], char *sep){

    char* ptr= (char*)malloc((10*sizeof(char))+1);
        strncpy(ptr, buf, 10);
        *(ptr+10)='[=11=]';
    int num_f=0;

    while((fields[num_f] = strtok(ptr,sep)) != NULL ){
        ptr = NULL;
        num_f++;
    }
    return num_f;
}


char* timeConversion(char* s) {
    char *fields[3];
    int num_f=0;
    char *ptr = (char*) malloc(100*sizeof(char));
    int hour=0;

    get_tokens(s, fields, ":");
    if(strstr(s,"PM")){
        hour=atoi(fields[0])+12;
    }
    else{
      hour=atoi(fields[0]);
    } 

    snprintf(ptr, 9, "%d:%s:%s" ,hour,fields[1],fields[2]);
    return ptr;
}

int main() {
    char* s = (char *)malloc(100 * sizeof(char));
    scanf("%s", s);
    char* result = timeConversion(s);
    printf("%s\n", result);
    return 0;
}

我在 timeConversion 函数 returns 之后看到一个 "stack smash"。我知道代码的逻辑有效，但无法弄清楚堆栈粉碎。

Answer 1

函数 get_tokens 使用 3 个指针写入 stack-allocated 缓冲区，而不检查缓冲区容量限制。 fields[num_f] = strtok 可能会导致堆栈缓冲区溢出。同时 fields 项可以在未初始化的情况下使用。

还存在内存泄漏，因为您从未 free 分配内存。

C 函数中的堆栈粉碎 return

Stack smashing in C on function return

c

pointers

char

stack-smash