Bug 930

trace:file:CreateFileW returning 0x64
trace:file:ReadFile 0x64 0x7e429764 36 0x7e4296e8 (nil)
err:cabinet:FDICopy FDIIsCabinet failed.

Код, возвращающий ошибку:
  /* check if it's really a cabfile. Note that this doesn't implement the bug */
  if (!FDI_read_entries(hfdi, cabhf, &fdici, &(CAB(mii)))) {
    ERR("FDIIsCabinet failed.\n");
    PFDI_CLOSE(hfdi, cabhf);
    return FALSE;
  }

Проверка проваливается, если cab-файл не содержит файлов или каталогов. А исследуемый файл весит всего 36 байт.

Created attachment 213 [details]
ci-temp0.cab

Подстановка нативного cabinet.dll позволяет обойти проблему маленького cab-файла.
Впоследствии может возникнуть проблема с кодировками имён файлов. Системная кодировка не должна быть KOI8 (проверено - на CP1251 работает).

Когда мы будем писать комментарии, в которых будет суть проблемы?
Один из файлов в архиве содержит в имени знак № (номер), и установка не сможет пройти в кодировке локали, которая не имеет такого знака (например, с кодировкой koi8-r).

По коду проверки CAB-файла - нужен патч, отключающий return FALSE, с сохранением сообщения об ошибке.

Почему разборки с багами начинаются, и не заканчиваются?
Надо уж написать тест красивый, отправить в wine-patches,
проблему исправить...

Надо исправить окончательно к релизу.

Как я понял, FDICopy вызвает функцию FDI_read_entries, в которой и происходит ошибка.
А ошибка в следующем:
Где-то в FDI_read_entries (fdi.c:565) идёт проверка на то сколько директорий содержит cab-файл. Если не содержит ни одной, то возвращает FALSE.
Далее FDICopy видит, что проверка не выполнилась, и тоже возвращает FALSE.
А Windows в этом случае возвращает TRUE

Проверил: могут быть такие cab-файлы, которые содержат файлы, но не содержат директории (бага 1086 - тестовый файл test6.cab)

Очевидно, что проверку на отсутствие директорий надо просто выкинуть.
При этом функция FDI_read_entries() или вызывающая её FDICopy() может работать с ошибками. Надо проверить

Проверил: Если в cab-файле нету папок, но есть, хотя бы один файл, то комманда 
num_folders = EndGetI16(buf+cfhead_NumFolders);
Всё-равно находит 1 директорию.

В случает, если в cab-файле нет ни папок, ни файлов, то данная комманда говорит, что директорий 0.

Думаю, что в данном случае, не следует удалять проверку на количесво директорий, а сделать так, чтобы она возвращала, например, EMPTY_CABINET.
Далее FDI_Copy смотрит возвращаемое значение, и если оно EMPTY_CABINET, то просто возвращает TRUE.

Таким образом, если папок 0, значит сab-файл пустой. Значит можно сразу возвратить TRUE (т.к он правильный, но нечего распаковывать)

Проблемы могут возникнуть в том случае, когда cab-файл битый, а мы будем считать его правильным.
Но думаю, что это не произойдёт, ведь до проверки на наличие папок стоит ещё несколько проверок.

Исправил. Тест проходит.

Ещё несколько функций используют FDI_read_entries(). И если она возвратит EMPTY_CABINET, то они будут считать это как TRUE.

Попробовал убрать в FDICopy() проверку на EMPTY_CABINET (т.е FDICopy считает, что  
FDI_read_entries() возвращает TRUE, и идёт дальше. В результате возникает ошибка доступа.

Получается, что просто TRUE или EMPTY_CABINET возвращать нельзя. Иначе придётся переписывать и другие функции, чтобы они тоже ловили EMPTY_CABINET.

Попробовал просто убрать проверку на наличие директорий и файлов.
В итоге FDI_read_entries() завершается без ошибок, а ошибка возникает в FDICopy() при чтении папок

Нет, всё-таки чтение папок и файлов проходит нормально, а ошибка возникает при чтени файлов в выражении:
for (file = CAB(firstfile); (file); file = file->next) { .... 

Заполнение структуры pfdici происходит в конце функции FDI_read_entries().

Поэтому, если количество папок=0, то возвращать EMPTY_CABINET надо не сразу после проверки на количество папок, а в самом конце, перед выходом. Тогда структура pfdici будет нормально заполнена.

Ещё можно добавить проверку на ситуацию когда папок не 0, а файлов 0. Такого быть  не может. 

Думаю, что бага исправлена.
Патч находится в wine-etersoft-public/patches/check/dlls_cabinet_empty-cab.patch

Ты не можешь возвращать из FDI_read_entries
TRUE, FALSE, EMPTY_CABINET
надо 3 самостоятельные константы.

И нельзя ли всё-таки обустроить так, чтобы FDI_read_entries возвращала TRUE/FALSE, а дальше с 0 количеством файлов разборка шла как-то общим порядком (без особого исключения для 0 количества файлов)

Согласен, что мой вариант не особо красивый. Но возвращать какие-то константы вместо TRUE и FALSE нельзя: ведь другие функции, использующие FDI_read_entries() ожидают именно TRUE или FALSE.
Но всё должно работать потому, что EMPTY_CABINET != 0 , а следовательно другие программы считают его за TRUE.

Переписал.
Теперь при пустом cab-файле FDI_read_entries() возвращает TRUE.
При этом пришлось поправить FDICopy(), так как она при этом падала

Итак, каков результат по данной баге?

Всё должно работать, надо только проверить в "1С Универсал".
Последнюю версию патча я, вроде, не закоммитил. Надо проверить

в текущей версии патч отсутствует.
Сделал патч для git. Осталось только проверить 1С Универсал

Будет в сборке -eter12

commit 5e3b980bb4d0079b3472617605cc128367f1c513
Author: Vitaly Perov <vitperov@etersoft.ru>
Date:   Sun Jun 22 14:10:02 2008 +0400

    cabinet: fix bug #930: empty *.cab is count as invalid

(In reply to comment #24)
> Будет в сборке -eter12
> 
> commit 5e3b980bb4d0079b3472617605cc128367f1c513
> Author: Vitaly Perov <vitperov@etersoft.ru>
> Date:   Sun Jun 22 14:10:02 2008 +0400
> 
>     cabinet: fix bug #930: empty *.cab is count as invalid
> 

Проверил на последней бете - не работает

Конфигурация для тестирования установки:
/var/ftp/pvt/Windows/1C/2008-07/universal-080707-full.exe

Created attachment 555 [details]
?1

И снова эти квадратики...

Created attachment 556 [details]
?2

Вылет

Такого файла в директории нет.

Установку нельзя запускать в локали koi8-r (в этой кодировке нет символа №)
запускай через
$ LANG=ru_RU.CP1251 wine

Да -так всё работает. сборка wine-1.0.9-alt15