试图将 char 数组复制到结构崩溃

Question

（作为序言，我的 C 很糟糕）

我正在尝试将字符串从 iOS 发送到 BLE 设备。我用swift对字符串进行编码并这样写：

func sendUserName(userName: String) {
    let bytes: [UInt8] = Array(userName.utf8)

    print(bytes.count)

    let data = NSData(bytes: bytes, length: bytes.count)
    capsenseLedBoard!.writeValue(data, forCharacteristic: userIdCharacteristic, type: CBCharacteristicWriteType.WithResponse)
}

我发送这个字符串"THISISATEST123456789"，这行print(bytes.count) 打印出20。

---

我像这样在 BLE 设备上接收数据并将其传递给下面的 userDidConnect 函数：

userDidConnect((char *)wrReqParam->handleValPair.value.val);

---

我有一个名为Event 的struct，看起来像这样：

struct Event {
    char time[20];   // The time in ISO 1601 format
    char name[3];    // The two character name of the event. See header for declarations.
    char userId[20]; // The userId of the connected user if one is present.
    struct Event* next;
};

---

我有一个这样声明的全局变量：

char currentlyConnectedUserID[20];

---

然后我有一个如下所示的入队函数：

/**
 Creates a new Event and adds to the linked list.

 @param time   The time in ISO 8601 format.
 @param name   The name descriptor of the event ("VS", "VO", etc.)
 @param userId The id of the user who is currently connect (if they are connected).
 */
    void enqueueEvent(char time[20], char name[3], char userId[20]) {

    struct Event* temp = (struct Event*)malloc(sizeof(struct Event));
    strncpy( temp->time,   time, 20);
    strncpy( temp->name,   name, 3);
    strncpy( temp->userId, userId, 20);
    temp->next = NULL;

    if(front == NULL && rear == NULL) {
        front = rear = temp;
        return;
    }
    rear->next = temp;
    rear = temp;
}

---

我有一个函数，它接受一个新的 userId，然后创建一个新的 Event 并将其添加到链表中。这就是它应该做的：

void userDidConnect(char *userId)
{
    size_t destination_size = sizeof(userId);
    snprintf(currentlyConnectedUserID, destination_size, "%s", userId);
    //enqueueEvent("2007-03-01T13:00:20", "UC", currentlyConnectedUserID);

    showMessageInUART(currentlyConnectedUserID, sizeof(currentlyConnectedUserID));
}

---

目前，如果我运行上面的userDidConnect 方法，我可以正确打印出currentlyConnectedUserID。但是，如果我取消注释掉这一行：

//enqueueEvent("2007-03-01T13:00:20", "UC", currentlyConnectedUserID);

我遇到“崩溃”。我在一个相当晦涩的 IDE（赛普拉斯的 PSoC Creator）中执行此操作，因此我看不到任何错误日志或 IDE 崩溃日志。我能说的唯一方法是永远不会调用showMessageInUART，所以我知道它必须是那条线。

如果我这样做，我可以成功创建一个新事件并将其加入队列：

enqueueEvent("2007-03-01T13:00:20", "UC", "1234567891234567891"); 

我唯一的想法是数组的大小可能是错误的？也许？或者也许有一些尾随的\0 把事情搞砸了？

---

建议更新：

我试过这样做：

size_t destination_size = strlen(userId) + 1;

这为currentlyConnectedUserID 提供了正确的值，但是排队仍然会导致崩溃。

--

我已将strcpy 替换为strncpy，这仍然会导致崩溃；(

--

试过这个以确保我没有溢出但仍然没有工作：

sprintf(currentlyConnectedUserID, "%.19s", userId);

---

更新

由于没有断点，我将队列更新为如下所示：

void enqueueEvent(char time[20], char name[3], char userId[20]) {

    UART_UartPutString("start enqueue");

    struct Event* temp = (struct Event*)malloc(sizeof(struct Event));

    UART_UartPutString("1");

    strncpy( temp->time,   time, 20);

    UART_UartPutString("2");

    strncpy( temp->name,   name, 3);

    UART_UartPutString("3");

    strncpy( temp->userId, userId, 20);

    UART_UartPutString("4");

    temp->next = NULL;

    UART_UartPutString("5");

    if(front == NULL && rear == NULL) {
        front = rear = temp;
        return;
    }
    rear->next = temp;
    rear = temp;
}

这条线崩溃了：

strncpy( temp->time, time, 20);

我们从来没有在这里成功过：UART_UartPutString("2");

如果我从 main 调用相同的函数，它工作正常。知道为什么从不同的方法调用它会在这里崩溃吗？

Answer 1

strcpy 函数将源指向的 C 字符串复制到目标指向的数组中，包括终止空字符（并在该点停止）。 因此，我建议您更改enqueueEvent 功能，使用strncpy 而不是危险的strcpy 为：

void enqueueEvent(char time[20], char name[3], char userId[20]) {

    struct Event* temp = (struct Event*)malloc(sizeof(struct Event));
    strncpy( temp->time,   time,20);
    strncpy( temp->name,   name,3);
    strncpy( temp->userId, userId,20);
    temp->next = NULL;

    if(front == NULL && rear == NULL) {
        front = rear = temp;
        return;
    }
    rear->next = temp;
    rear = temp;
}

还将temp 指针的分配从enqueueEvent 函数的本地堆栈更改为全局级别，因为指针分配在离开函数时消失了。