Stlink: Компиляция с GCC 11 не выполняется

Созданный на 9 дек. 2020 · 7Комментарии · Источник: stlink-org/stlink

Я не могу собрать stlink 1.6.1 с компилятором GCC 11. Fedora GNU / Linux Rawhide.
Ошибки бывают:

/builddir/build/BUILD/stlink-1.6.1/src/st-util/gdb-server.c:90:22: error: 'stlink_open_usb' accessing 64 bytes in a region of size 28 [-Werror=stringop-overflow=]
   90 |                 sl = stlink_open_usb(st->logging_level, st->reset, serialnumber, 0);
      |                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/builddir/build/BUILD/stlink-1.6.1/src/st-util/gdb-server.c: In function 'serve':
/builddir/build/BUILD/stlink-1.6.1/src/st-util/gdb-server.c:90:22: note: referencing argument 3 of type 'char *'
In file included from /builddir/build/BUILD/stlink-1.6.1/include/stlink.h:297,
                 from /builddir/build/BUILD/stlink-1.6.1/src/st-util/gdb-server.c:27:
/builddir/build/BUILD/stlink-1.6.1/src/usb.h:71:15: note: in a call to function 'stlink_open_usb'
   71 |     stlink_t *stlink_open_usb(enum ugly_loglevel verbose, int reset, char serial[STLINK_SERIAL_MAX_SIZE], int freq);
      |               ^~~~~~~~~~~~~~~

In function 'do_connect',
    inlined from 'do_connect' at /builddir/build/BUILD/stlink-1.6.1/src/st-util/gdb-server.c:87:18,
    inlined from 'main' at /builddir/build/BUILD/stlink-1.6.1/src/st-util/gdb-server.c:212:10:
/builddir/build/BUILD/stlink-1.6.1/src/st-util/gdb-server.c:90:22: error: 'stlink_open_usb' accessing 64 bytes in a region of size 28 [-Werror=stringop-overflow=]
   90 |                 sl = stlink_open_usb(st->logging_level, st->reset, serialnumber, 0);
      |                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Пожалуйста, помогите мне это исправить.

bucompilation buneeds-fix olinux staturesolved

Источник

Vascom

Все 7 Комментарий

Может быть STLINK_SERIAL_MAX_SIZE define должен быть установлен в 28 ?

Vascom 9 дек. 2020

Вы можете исправить src/st-util/gdb-server.c :

static char serialnumber[28] = {0};

static char serialnumber[STLINK_SERIAL_MAX_SIZE] = {0};

Ant-ON 9 дек. 2020

🎉1

Я предпочитаю видеть исправления в апстриме.

Vascom 9 дек. 2020

@Vascom Я тоже. Но чтобы исправить это в апстриме, нужно проверить его работу.

ps Вы создали описание проблемы не по шаблону. @ Nightwalker-87 может удалить его. См. № 906

Ant-ON 9 дек. 2020

👀1

Да, это работает.

Шаблон не заботится о проблемах с компиляцией.

Vascom 9 дек. 2020

Шаблон предназначен для использования и наполнения контентом, который также включает описание. Так что подумайте об этом в следующий раз.

Nightwalker-87 9 дек. 2020

Вы можете исправить src/st-util/gdb-server.c :
static char serialnumber[28] = {0};
к
static char serialnumber[STLINK_SERIAL_MAX_SIZE] = {0};

Я почти предложил это (изначально я обнаружил проблему при тестировании gcc-11 с Fedora). Меня беспокоил этот цикл в gdb-server.c:
for (size_t k = 0; j> = 0 && k <sizeof (серийный номер); ++ k, j - = 2) {
char buffer [3] = {0};
memcpy (буфер, optarg + j, 2);
серийный номер [длина - k] = (uint8_t) strtol (buffer, NULL, 16);

На что повлияет изменение размера "серийного номера". Не зная намерения или кода в целом, я не чувствовал себя комфортно, рекомендуя изменять размер «серийного номера».

И чтобы дать людям немного предыстории. В gcc-11 добавлена диагностика для обнаружения выходов за допустимые пределы для аргументов массива. Проще говоря, если функция объявляет аргумент как массив с фиксированной длиной, то GCC предполагает, что на весь массив потенциально можно ссылаться.

stlink_open_usb объявляет свой третий аргумент как массив символов из 64 байтов, но «серийный номер» составляет всего 28 байтов, и GCC, естественно, предупреждает, потому что считает, что stlink_open_usb потенциально может выйти за пределы аргумента «серийный номер».