从函数

Question

我的问题是，就像我在标题上提到的那样，我有一个函数，在返回时会导致SIGABRT被引发。我在我的程序上运行了valgrind，并且在这一点上得到了这个。

==5807== Process terminating with default action of signal 6 (SIGABRT)
==5807==    at 0x52F5428: raise (raise.c:54)
==5807==    by 0x52F7029: abort (abort.c:89)
==5807==    by 0x53377E9: __libc_message (libc_fatal.c:175)
==5807==    by 0x53D911B: __fortify_fail (fortify_fail.c:37)
==5807==    by 0x53D90BF: __stack_chk_fail (stack_chk_fail.c:28)
==5807==    by 0x402E8B: foo (file.c:43)
==5807==    by 0x202C27323939312C: ???
==5807==    by 0x592D4D4D2D444426: ???
==5807==    by 0x66202C2927595958: ???
==5807==    by 0x2965736C60: ???
==5807==    by 0x505770F: ??? (in /usr/lib/x86_64-linux-gnu/libodbc.so.2.0.0)

==5807==

这是我执行的代码，堆栈错误导致add_user返回ret语句中止：

int add_user(SQLHDBC dbc, char * mail, char * password, char * name,  char * date) {

char query[TAM];
SQLHSTMT stmt;
SQLRETURN ret;

if (mail == NULL || password == NULL || name == NULL ||  date == NULL)
    return ERR;

sprintf(query, "INSERT INTO mms_user values (default,'%s',encrypt_password('%s'),set_type(),'%s',to_date('%s', 'DD-MM-YYYY'), false)", mail, password, name,date);

ret= DBExecuteQuery(dbc, query, &stmt);

DBFreeHandle(&stmt);
return ret;
}

int DBExecuteQuery(SQLHDBC dbc, char * query, SQLHSTMT *stmt) {

SQLRETURN ret;

/*Error control*/
if (query == NULL || stmt == NULL)
    return ERR;

/*Allocates memory for a new statement*/
ret = SQLAllocHandle(SQL_HANDLE_STMT, dbc, stmt);
if (!SQL_SUCCEEDED(ret)) {
    return ERR;
}

/*Executes query and stores result in stmt*/
ret = SQLExecDirect(*stmt, (SQLCHAR *)query, SQL_NTS);
if (!SQL_SUCCEEDED(ret))
    return ERR;

return OK;

}

int DBFreeHandle(SQLHSTMT * stmt){

 int ret;
  /*Frees allocated memory*/
  ret = SQLFreeHandle(SQL_HANDLE_STMT, *stmt);
  if (!SQL_SUCCEEDED(ret)) {
      return ERR;
  }

}

就我检查而言，该功能正常执行。我不知道会发生什么，如果你们中有人能提供任何可能的解释，我会非常感激。也可以随时询问更多信息（这是我已经教过的唯一有用信息，因此我可能需要一些指导）。

提前谢谢

Answer 1

此代码很危险：

char query[TAM];
 .
 .
 .
sprintf(query, "INSERT INTO mms_user values"
    " (default,'%s',encrypt_password('%s'),set_type(),"
    "'%s',to_date('%s', 'DD-MM-YYYY'), false)",
     mail, password, name,date);

你绝对没有边界检查，所以它会很容易地溢出你的堆栈。