C：一种更安全的方法来检查函数中的缓冲区并附加到它？

Question

我有一个函数，我需要返回另一个日志函数的时间，它看起来像这样：

//put time in to buf, format 00:00:00\0
void gettimestr(char buf[9]) {
  if(strlen(buf) != 9) { //experimental error checking
    fprintf(stderr, "Buf appears to be %d bytes and not 9!\n", strlen( buf ));
  }
  time_t cur_time;
  time(&cur_time);
  struct tm *ts = localtime(&cur_time);
  sprintf(buf, "%02d:%02d:%02d",
        ts->tm_hour,
        ts->tm_min,
        ts->tm_sec );
  strncat(buf, "\0", 1);
}

现在我猜主要问题是检查缓冲区是否足够长， sizeof() 返回一个指针大小，而 strlen 似乎是随机的在两个不同的调用中返回 0 或诸如 12 之类的值。

我的第一个问题是，我如何能够安全地检测缓冲区的大小，这可能吗？

我的另一个问题是，接受 buf[9] 是一种有利的方法，还是应该接受指向缓冲区的指针，并使用 strcat() 而不是 sprintf() 将时间附加到它？ sprintf 使向时间值填充零变得更容易，尽管它似乎只接受字符数组而不接受指针。

Answer 1

您的函数假设传入的缓冲区已包含一个包含 9 个字符的以 null 结尾的字符串。这没有道理。

正确的方法是请求大小作为参数：

void gettimestr(char *buf, int bufferSize) {

并使用 snprintf:

snprintf(buf, bufferSize, "%02dx....", ....);<sub>*</sub>

并终止字符串，因为如果超出限制，则 snprintf 不会执行此操作：

buf[bufferSize-1] = 0;

您可以这样调用您的函数：

char buffer[16];
gettimestr(buffer, sizeof(buffer));

没有其他方法可以确定大小。这不是 Java 中数组知道其大小的情况。传递 char * 只会向函数发送一个指针，而无需进一步的信息，因此获取缓冲区大小的唯一方法是要求调用者指定它。

（编辑：snprintf 应始终正确终止字符串，如注释中所指出的。）

Answer 2

@EboMike 是对的。只是为了补充他的答案，您可以使用以下方法检查缓冲区：

void gettimestr(char *buf, int bufferSize) {
    if (!buf) {
        fprintf(stderr, "Null buffer\n");
        return;
    }

 // rest of the code

 }