Question

我做了这个功能：

void procesar_llamadaAFuncion(t_proceso *unProceso, char *sentencia){
    char *nombreFuncion = sentencia;
    char *nombreFuncionSinParentesis = NULL;

    string_trim(&nombreFuncion);
    nombreFuncionSinParentesis = malloc(sizeof(char)*(strlen(nombreFuncion)-2));
    strncpy(nombreFuncionSinParentesis, nombreFuncion, strlen(nombreFuncion)-2);

    puts(nombreFuncionSinParentesis);

    push_stack(unProceso->pcb->seg_stack, nombreFuncionSinParentesis, unProceso->pcb->program_counter);

    unProceso->pcb->program_counter = get_pos_funcion(unProceso->pcb->funciones, nombreFuncionSinParentesis);

    free(nombreFuncion);
    free(nombreFuncionSinParentesis);

t_proceso是什么并不重要，问题是这个函数接收一个字符数组。

函数将接收它的字符数组总是“something（）”，我想要做的是删除最后两个字符“（）”，然后调用函数push_stack（）。

问题在于，当我运行Valgrind时，我得到了这个：

==17129== Invalid read of size 1
==17129==    at 0x4C2BFD4: __GI_strlen (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==17129==    by 0x50BFCEB: puts (ioputs.c:37)
==17129==    by 0x403D30: procesar_llamadaAFuncion (proceso.c:455)
==17129==    by 0x40313D: procesar_siguiente_instruccion (proceso.c:132)
==17129==    by 0x404B1A: probarProcesos (test.c:83)
==17129==    by 0x404C7F: main (test.c:111)
==17129==  Address 0x5436da8 is 0 bytes after a block of size 8 alloc'd
==17129==    at 0x4C2B6CD: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==17129==    by 0x403CDF: procesar_llamadaAFuncion (proceso.c:452)
==17129==    by 0x40313D: procesar_siguiente_instruccion (proceso.c:132)
==17129==    by 0x404B1A: probarProcesos (test.c:83)
==17129==    by 0x404C7F: main (test.c:111)

我不知道我做错了什么，所以任何帮助都会受到赞赏。

Answer 1

这是因为strncpy不会终止目标字符串：

将源的前几个num字符复制到目标。如果在复制num个字符之前找到源C字符串的结尾（由空字符表示），则使用零填充目标，直到已经写入总数为num个字符。

如果source长于num，则不会在目标的末尾隐式附加空字符（因此，在这种情况下，目标可能不是空终止的C字符串）。

这应解决问题：

size_t nobmreLen = strlen(nombreFuncion)-2;
// Don't forget to add +1 for the null terminator
nombreFuncionSinParentesis = malloc(sizeof(char)*(nobmreLen+1));
strncpy(nombreFuncionSinParentesis, nombreFuncion, nobmreLen);
nombreFuncionSinParentesis[nobmreLen] = '\0';

Answer 2

您必须null终止字符串。这对于参数“sentencia”和删除括号

的结果都很重要

Answer 3

 nombreFuncionSinParentesis = malloc(sizeof(char)*(strlen(nombreFuncion)-2));

以上不正确，您还需要为空终止分配空间。我不明白-2的作用，但是为你正在尝试的任何内容添加+1个字节，即

malloc(sizeof(char)*(strlen(nombreFuncion)-2 + 1));

正如评论中所建议的那样，不应该使用strncpy，它是为古老版本的unix的特定需求而编写的一个模糊函数。 Read this post或者确实很好的article发布了对另一个答案的评论。

free(nombreFuncion);

以上内容非常可疑，您正在执行free(sentencia)，这是在函数外部分配的。如果这是意图，你应该考虑更好的程序设计。

使用strncpy。 Valgrind抛出无效的读数

3 个答案: