在 C 中使用 printf 不安全吗

Question

我是 C 的新手，所以我开始玩一些代码。我的代码中有一个错误，因为我认为使用 printf(pass) 不安全，因为它可以覆盖该值，因此对用户不安全。我想知道printf(pass) 在我的代码中是不安全的吗？另外，我怎样才能让用户在不更改代码的情况下打印消息finally logged in。有什么办法吗？

我的代码：

#include <stdio.h>

char pass[100];

char getPass() {
  int value = 'G';
  int * j = & value;
  fgets(pass, sizeof(pass), stdin);
  printf("your entered pass is ");
  printf(pass);
  return (char)( * j);
}

void main() {
  printf("enter the pass here ");
  if (getPass() == 'K') {
    printf("finally logged in\n");
    exit(0);
  } else {
    printf("Wrong password\n");
    exit(1);
  }
}

Answer 1

如果函数fgets调用成功，它会提供一个可以输出的字符串。

如果这样输出得到的字符串会出现问题

printf(pass);

并包含转换指定。

所以不是那个电话

printf(pass);

使用起来更安全

printf( "%s", pass );

或

puts(pass);

初步可以通过fgets的调用删除字符串中存储的换行符。

Answer 2

是的，它不安全，但不是因为您建议的原因。如果您查看man 页面中的printf() 函数，您会看到它具有以下原型：

int printf(const char * restrict format, ...);

应用于第一个参数的const修饰符指定调用此函数时不会更改此参数的值。

但是，您还会注意到此参数称为format。那是因为它应该指定一个格式字符串。在您的程序中，没有什么可以阻止用户输入像%p 这样的格式说明符，在这种情况下，您对printf() 的调用将开始打印出堆栈的内容。有一个Wikipedia article 更详细地描述了这个漏洞。