Использование sizeof в C

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).

Затем вы отформатируете три целых числа в шестисимвольный формат с шестью символами, для чего потребуется хранить 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).

Это все еще не сработает, но оно не сработает. Другие ответы объясняют, как правильно использовать функциональность.

(О, и не использовать получает() тоже. Просто потому, что в стандартной библиотеке не означает, что это хороший код.)

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; 
}