在C中使用sizeof

Question

我有以下程序，它正在崩潰。有人知道它爲什麼會崩潰嗎？

/* writes a, b, c into dst 
** dst must have enough space for the result 
** assumes all 3 numbers are positive */ 
void concat3(char *dst, int a, int b, int c) { 
    sprintf(dst, "%08x%08x%08x", a, b, c); 
} 

/* usage */ 
int main(void) { 
    printf("The size of int is %d \n", sizeof(int)); 
    char n3[3 * sizeof(int) + 1]; 
    concat3(n3, 0xDEADFACE, 0xF00BA4, 42); 
    printf("result is 0x%s\n", n3); 
    return 0; 
} 

Answer 1

你混淆了二進制數據的大小（這是什麼sizeof）給你，用文字表述的十六進制的大小，這是你想要什麼店。

在大多數當前系統上，sizeof(int)的計算結果爲4.因此，緩衝區n3將能夠存儲13個字符（3 * 4 + 1 == 13）。

然後，您將三個整數格式化爲8字符的十六進制格式，這需要3 * 8 + 1 == 25個字符來存儲。由此產生的緩衝區溢出導致崩潰。

數據類型int的大小應該很明顯，當您將其格式化爲文本（並且自己指定字段寬度！）時。

Answer 2

它正在崩潰，因爲sizeof(int)（最有可能在您的系統上）4，這意味着n3的長度爲13個字節。然後嘗試將8 + 8 + 8 = 24個字符寫入它。

Answer 3

我真的不明白sizeof在代碼中有什麼要做的。在concat3中，您試圖將每個提供的整數的文本表示形式打印爲8個字符的十六進制字符串：因此所需的緩衝區大小應該等於8 * 3 + 1 = 25，而sizeof(int)與此無關。

你似乎在用int來混合內存中佔用的大小，以及它的文本表示的長度（在你的情況下它很容易確定，因爲它是由你的sprintf格式字符串修復的）。

附註：sprintf是一個真正不安全的函數，您應該考慮棄用。

Answer 4

使用snprintf代替sprintf。想想小貓！

但嚴重的是，你不應該創建接口與緩衝區指針，但沒有長度信息。 concat應該有一個最大長度參數。然後在裏面使用snprintf。 concat的長度是sizeof（n3）。

它仍然無法工作，但它也不會崩潰。其他答案解釋瞭如何獲得正確的功能。

（呵呵，不要使用gets（）兩種。正因爲它是在標準庫並不意味着它是很好的代碼。）

Answer 5

嘗試3*2*sizeof(int)+1，其中2*sizeof(int)是所需的字節數以十六進制打印int的每個字節。當然，因爲您使用的格式爲%08X，並且期望得到固定寬度的結果，所以您確實應該使用uint32_t。順便說一句，您的程序也錯誤地將0xDEADBEEF作爲int，它可能不適合，從而進入實現定義的轉換爲簽名類型的領域。

下面是這些更正一個版本：

#include <inttypes.h> 
#include <stdio.h> 

/* writes a, b, c into dst 
** dst must have enough space for the result 
** assumes all 3 numbers are positive */ 
void concat3(char *dst, uint32_t a, uint32_t b, uint32_t c) { 
    sprintf(dst, "%08"PRIX32"%08"PRIX32"%08"PRIX32, a, b, c); 
} 

/* usage */ 
int main(void) { 
    printf("The size of int is %d \n", sizeof(int)); 
    char n3[25]; 
    concat3(n3, 0xDEADFACE, 0xF00BA4, 42); 
    printf("result is 0x%s\n", n3); 
    return 0; 
}