Xpeccy

[Q-Master]

При сборке

Linking CXX executable xpeccy
CMakeFiles/xpeccy.dir/src/libxpeccy/filetypes/rzx.c.o: In function `rzxLoadFrame':
rzx.c:(.text+0x804): warning: the use of `tmpnam' is dangerous, better use `mkstemp'

В целом все ок. Я не заметил проблем с fdi.c.

[Advertiser]

[SAM style]

При сборке

Linking CXX executable xpeccy
CMakeFiles/xpeccy.dir/src/libxpeccy/filetypes/rzx.c.o: In function `rzxLoadFrame':
rzx.c:(.text+0x804): warning: the use of `tmpnam' is dangerous, better use `mkstemp'

В целом все ок. Я не заметил проблем с fdi.c.

А теперь главный вопрос - загрузка fdi нормально работает? Хотя бы тех, что нормально работали до этого.
Впрочем, я уже перестраховался и сделал свои swap16 и swap32. Оно, конечно, где-то в byteswap.h есть аналогичные, но это уже мелочь.

А насчет tmpnam vs mkstemp - мне tmpnam удобнее, т.к. ему не нужен образец пути и он не зависит от оси.

[ram_scan]

А насчет tmpnam vs mkstemp - мне tmpnam удобнее, т.к. ему не нужен образец пути и он не зависит от оси.

tmpnam недостаточно секурное.

[SAM style]

tmpnam недостаточно секурное.

Разве что из-за этого:

Вызывать функцию tmpnam можно несколько раз, но не более чем TMP_MAX (обычно TMP_MAX = 238328) раз.

Но это надо сильно постараться... Используется только для rzx, чтобы вытаскивать оттуда снапшоты, а потом передать путь в функцию загрузки снапшота и потом удалить временный файл.

[ram_scan]

Разве что из-за этого:Но это надо сильно постараться... Используется только для rzx, чтобы вытаскивать оттуда снапшоты, а потом передать путь в функцию загрузки снапшота и потом удалить временный файл.

Не. tmpnam создает только имя. За время между рожанием имени и открытием файла другая программа может подменить путь или пошаманить с символической ссылкой, и файл попадет не туда куда было задумано. Еще с переполнением буфера там залепуха есть.

[SAM style]

Не. tmpnam создает только имя. За время между рожанием имени и открытием файла другая программа может подменить путь или пошаманить с символической ссылкой, и файл попадет не туда куда было задумано. Еще с переполнением буфера там залепуха есть.

Окааай, а если я сгенерю имя себе в sname[L_tmpnam] и добавлю к нему суффикс "-xpeccy.tmp", то другие процессы его у меня отнять не должны, так?
Просто надо на C получить уникальное имя файла в месте, где 100% есть возможность записи, желательно ОС-независимым методом (без перебирания TMP, TEMP, TMPDIR...)

[Q-Master]

А теперь главный вопрос - загрузка fdi нормально работает? Хотя бы тех, что нормально работали до этого.
Впрочем, я уже перестраховался и сделал свои swap16 и swap32. Оно, конечно, где-то в byteswap.h есть аналогичные, но это уже мелочь.

fdi я не проверял. У меня они не факт что есть. А вот трд нормально загружаются.

А насчет tmpnam vs mkstemp - мне tmpnam удобнее, т.к. ему не нужен образец пути и он не зависит от оси.

Это не ко мне. Это к цомпилятору.

[SAM style]

fdi я не проверял. У меня они не факт что есть. А вот трд нормально загружаются.

Для trd не нужна смена порядка байт. Вот, например, fdi ZXFormat#1 - тыц.

[SAM style]

Хм... 1000 get!

[Q-Master]

Для trd не нужна смена порядка байт. Вот, например, fdi ZXFormat#1 - тыц.

Забавно, но в режиме pentevo диск не видится вообще, а в режиме пентагона все ОК. Все загрузилось и заиграло.