C sprintf 覆盖？

Question

谁能解释一下为什么这不起作用？看来程序停在 sprintf(t->tuple[0], "abc");我不知道为什么。真的需要帮助。

int testRemoveMeio(){

  int result, i;
  struct list_t *list = list_create();

  char *tdata[3] = {"   ", "2014", "Fixe!"};
  struct tuple_t *t = tuple_create2(3, tdata);
  struct tuple_t *tdups[4];

  struct entry_t *entries[4];

  sprintf(t->tuple[0], "abc");
  tdups[0] = tuple_dup(t);
  entries[0] = entry_create(tdups[0]);
  list_add(list, entries[0]);

  //extra code similar to above
  return 0;
}

编辑：

struct tuple_t {
    int tuple_dimension; 
    char **tuple;        
};

struct tuple_t *tuple_create2(int tuple_dim, char **tuple){

    struct tuple_t *t = (struct tuple_t *) malloc(sizeof(struct tuple_t));

    if(t == NULL)
        return NULL;

    t->tuple_dimension = tuple_dim;
    t->tuple = tuple;

    return t;
}

编辑添加来自 cmets 的另一个示例：

#include <stdio.h>
#include <string.h>
int main(void)
{
  char * s = "ABC";
  printf("%s\n",s);
  sprintf(s,"DEF");
  printf("%s\n",s); 
  return 0;
}

Answer 1

我将您评论中的代码粘贴到问题中。我认为，这和您的原始代码都失败了，原因相同：您正在修改字符串文字。

在 C 语言中，可悲但真实的是，像 "hello world" 这样的字符串文字是非法修改的，但可以在没有 const 和没有强制转换的情况下分配给 char*。这很容易搞砸，因为修改这样的字符串是未定义的行为，因为它可能驻留在程序的只读内存中。

要修复它，试试这个：

char tdata[3][6] = {"   ", "2014", "Fixe!"};

或者在你的第二个例子中：

char s[] = "ABC";

通过这种方式，您可以分配一个具有可写存储空间的实际字符数组。但是，请始终使用snprintf() 而永远不要使用sprintf()，因为后者容易出现缓冲区溢出，这可能会使您的程序崩溃或更糟。

Answer 2

你的第二个例子是错误的。您应该在 char 的某些 array 上使用 snprintf(3)（没有理由在 2014 年使用 sprintf；它既古老又危险，因为它可能会给出 buffer overflow）像这样：

#include <stdio.h>
int main(void) {
  char buf[64];
  snprintf(buf, sizeof(buf), "here %s", "ABC");
  printf("%s\n",buf); 
  return 0;
}

您可以使用一些动态分配的内存区域（但是，您应该知道它的大小），例如

#include <stdio.h>
#include <stdlib.h>
int main(void) {
  size_t siz=32;
   char* ptr = malloc(siz);
   if (!ptr) { perror("malloc"); exit(EXIT_FAILURE); };
   snprintf(ptr, siz, "here %s and two is %d", "ABC", 2);
   printf("%s\n", ptr);
   free (ptr);
   return 0;
}

如果您的系统提供它（例如在 Linux 上），您可以使用 asprintf(3)

#define _GNU_SOURCE
#include <stdio.h>
int main (void) {
  char* ptrzone = NULL;
  asprintf(&ptrzone, "and here %s and one is %d", "ABC", 1);
  printf("%s\n", ptrzone);
  free (ptrzone), ptrzone = NULL;
}

在严肃的程序中，你经常应该使用snprintf的结果。

阅读您正在使用的所有函数的文档，同时阅读：perror(3)、exit(3)、malloc(3)、free(3)

你绝对应该阅读更多关于C dynamic memory allocation的信息

您应该使用所有警告和调试信息 (gcc -Wall -g) 进行编译，并学习使用调试器 (gdb) 和内存泄漏检测器 valgrind

Answer 3

缺少一些重要信息，如 struct tuple_t 的定义和函数 tuple_create2 的定义。如果错误发生在 sprintf(t->tuple[0], "abc") 上，则 t->tuple 和/或 t->tuple[0] 未正确初始化/分配。对于示例来说，前者应该是一个至少为 1 的数组（但如果我们查看 tdata，则更可能是 3 个）字符串，而后者应该是一个字符串缓冲区（char 数组）。检查您或向我们提供tuple_create2 功能。

使用 sprintf 复制字符串的方式很危险。在您的示例中这有效，但如果要复制的字符串包含 '%' 字符，则可能会发生坏事（% 是用于在 printf/scanf 系列函数中指定格式的字符。如果您想继续使用 @987654330 @ 使用sprintf(t->tuple[0], "%s", "abc") 但更简单的是strcpy(t->tuple[0], "abc")

既然你现在已经添加了tuple_create2的定义，我试着回答一下。

在 C 中，由您来管理内存。你对tuple_create2 有什么期望？您的代码为“标题”（即包含维度和字符串数组的struct tuple）分配空间并用tuple 填充它。结果是一个新对象，它指向数据的相同内存（即字符串数组）。这没关系，除非您想修改此数组（似乎是这种情况）。有两件事：数组本身（每个条目都是一个指向字符数组的指针 - 你想修改指针吗？）和字符串（字符数组的内容）。有几种可能性，但这取决于你。例如，函数可以分配自己的指针数组并使用现有数据进行初始化：

struct tuple_t *tuple_create2(int tuple_dim, char **tuple){

    struct tuple_t *t = (struct tuple_t *) malloc(sizeof(struct tuple_t));

    if(t == NULL)
        return NULL;

    t->tuple_dimension = tuple_dim;
    t->tuple = (char **) malloc(sizeof(char *) * tuple_dim);

    for(i = 0; i < tuple_dim; i++)  // could be replaced by a memcpy
      t->tuple[0] = tuple[0];  

    return t;
}

那么你必须知道我们只复制了指针。所以当你想修改一个元素时，你只需复制指针：

 t->tuple[0] = "abc";    

或者你分配一个新的字符串，如果你认为它可以被修改并且你不想要这个。我给你一个“abc”的例子，但这不相关，因为“abc”是一个不会被修改的常量字符串。无论如何，这里是代码（注意使用stdrup，它复制了执行malloc的字符串）

t->tuple[0] = strdup("abc");