结构中的strcpy提供了SIGABRT

Question

我目前正在使用C语言编写的FTP客户端，它的运行情况还不错。 我成功编写了一个连接到FTP服务器并使用用户名和密码登录的函数，但是返回错误。 我已经设置了一个struct FTPError {}; 具有3个字段：

1. int错误代码
2. int FTP错误域（特定于我的函数）
3. char[256]用户可读的描述

函数的调用者通过引用该函数来传递结构，然后用数据填充它。 但是我正在努力填充用户可读的字符串（ char[256] ）。 我用strcpy填充字符串，但是当我调用它时，我的程序会发出SIGABRT信号。 我向您展示我的代码的简化部分：

struct FTPError {
    int status;
    int domain;
    char message[FTP_ERROR_MAX];
};

typedef int FTPConnection;

FTPConnection FTPConnect(const char *hostname, const char *username, const char *password, struct FTPError *errn) {

    int socket = /* socket file descriptor */

    // Success
    if(success == 1) {
        if(errn) {
            // when I comment out the line below, no signal is sent
            strcpy(errn->message, "User successfully loged in");
            errn->status = 230;
            errn->domain = kServerReplyDomain;
        }
    }

    // return the file descriptor
    return sockfd;
}

PS：这是Xcode在错误控制台中给我的：

Program loaded.
run
[Switching to process 2566]
Running…
SOCK: 3
Program received signal:  “SIGABRT”.
sharedlibrary apply-load-rules all
kill
quit

0x00007fff824c03cc  <+0000>  mov    $0x2000025,%eax
0x00007fff824c03d1  <+0005>  mov    %rcx,%r10
0x00007fff824c03d4  <+0008>  syscall 
0x00007fff824c03d6  <+0010>  jae    0x7fff824c03dd <__kill+17> --> (points this line) 
0x00007fff824c03d8  <+0012>  jmpq   0x7fff82560a8c <cerror>
0x00007fff824c03dd  <+0017>  retq 

3 __kill
2 __abort
1 __stack_chk_fail
0 main

PPS：我被要求显示调用该函数的代码：

int main (int argc, const char * argv[]) {
    struct FTPError reply;
    FTPConnection socket;

    socket = FTPConnect("ftp.belnet.be", "anonymous", "pwd", &reply);

    printf("SOCK: %d\n", socket);

    return 0;
}

Answer 1

给函数的errn指针很可能没有正确初始化，否则无效。 否则， FTP_ERROR_MAX也可能太小，因此strcpy()会产生缓冲区溢出。

Answer 2

您确定要传递内存分配的结构而不是未分配的指针吗？ 调用应类似于：

FTPError myError;
FTPConnect(host, user, password, &myError);

否则，如果您更喜欢使用指针：

FTPError* myError = (FTPError*) malloc(sizeof(FTPError));
FTPConnect(host, user, password, myError);

Answer 3

从它说的内容来看，我猜想您是用其他一些代码破坏了堆栈，并且它只是在此处显示。

无论如何，要确保您应该始终使用类似以下内容来初始化errn结构

 struct FTPError reply = { .status = 0 };

Answer 4

尝试使用保存strcpy变体：

strncpy(errn->message, "any string with unknown length", FTP_ERROR_MAX-1 );
errn->message[ FTP_ERROR_MAX-1 ] = 0;