ImageUtils

Цитата:

---

Сообщение от vva

Известная ошибка, многие "нарывались".

---

В данном случае - это мой недосмотр. Метод, для которого используется этот байтовый счётчик, принимает на вход 16-ти битное слово, то есть при его написании я этот момент учёл, а вот в цикле.. Прохлопал :) Думаю включить проверку переполнения и протестировать в таком варианте программу - посмотрим, где ещё вылезет :) Но это точно в выходные, в оставшиеся два дня особо времени нет..

- - - Добавлено - - -

В целом же, после фикса этой ошибки ушло так же некоторое количество ошибок на образах - типа - что-то там не так с ФС :) Но поскольку на внешний взгляд файлы выглядили более менее..

Хотя то, что я получил некоторое количество ошибок на ISO образ с ФС RSX, взятый с bitsaver - должно было насторожить :) Но.. Решил, что криво образ сделали :) Но, в конце концов, собственно на нём я и воевал с этой ошибкой. И всё в порядке с ним :)

На нём лежит некоторая кучка .dsk файлов - их я тоже включил в автоматическое тестирование - надо будет посмотреть - что там программа про них в лог накарябала :)

- - - Добавлено - - -

В тестировании сейчас участвует 61 образ общим размером 1.3 Гб, в которых находится 2497 каталогов и 236766 файлов, общим размером 2,26 GB (для RSX файлы могут сохраняться в нескольких вариантах, поэтому увеличение).
Время тестирования - примерно 21 минута :)

Всё таки не выдержал и врубил проверку арифметических переполнения и ... э.. underflow :)
Три ошибки уже поправил - две - подсчёт КС (ок) и проверку 31 бита (сдвиг, тоже ок)

- - - Добавлено - - -

И ещё одна КС и сдвиговые операции

Занимательный образ k.rar :)
Есть блок заголовка (и не один), который, с одной стороны, используется и (вроде как) имеет правильную КС, а с другой - одна из дат файла (дата последнего изменения) в нём неправильная :)
Похоже, придётся в описание правил преобразования из сырых данных в объекты .NET добавить для полей что-то типа флажка "Допустимы ошибки преобразования" :) Некоторая проблема в том, что - добавить легко, обрабатывать флаг тоже легко, а вот как сохранить информацию о том, что была ошибка.. Надо подумать :)

- - - Добавлено - - -

Первая часть реализована. Надо сохранением информации - думаю..

Первый вариант сохранения информации о проблемах при преобразовании сырых данных в заголовок сделан. Дальше посмотрю - нужно ли будет сохранять (и какую) доп информацию.

Добавил некоторое количество эвристик по решению - насколько (потенциальный) заголовок повреждён и стоит ли его (потом) как-то обрабатывать - на предмет восстановления данных

Среди образов, которые использовал для тестирования - образов с потерянными (нет ни в одном каталоге, заголовок правильный, файл не помечен на удаление) файлами нет. Надо будет сделать парочку образов для тестирования.

Следующие шаги - образы лент в формате BRU, а также - ODS-2 :) Вроде есть и ленты в формета BRU, снятые с ODS-2.

Буду по мере возможности писать код.

Пофиксил ещё четыре ошибки. Две в ODS (точнее говоря - одну я исправил в процессе рефакторинга, сам того не зная, но посадил другую) и две в RT-11 (опять спасибо включению проверки переполнения) - при преобразовании даты и выход в определённых сценариях при ошибках в ФС за границу образа.

По результатам последних доработок некоторые внутренние перетряски (изменение названий разного, влияние на функционал не оказывают)

И добавил заготовки под BRU и ODS-2. Пока практически пустые - некоторые наработки я начал (давно) делать, но там всё недоделанное и непроверенное. Скорей - по мотивам ODS-1 и поддержки лент что-то было добавлено и.. заброшено до сегодняшнего дня :)

Реализовал давно зревшую в голове мысль - теперь указание ФС - это не ключ, а аргумент, как и указание устройства.
Было

ImageUtils -rt11 -unpack file.dsk catalog rx50

Стало

ImageUtils -unpack file.dsk catalog rx50 rt11

или можно

ImageUtils -unpack file.dsk catalog rt11 rx50

Из плюсов
- теперь добавление новой ФС не требует доработки парсера командной строки
- порядок - устройств ФС или ФС устройства - роли не играет, главное, что бы были указаны верно
- построение командой строки теперь более одинаково для команд -pack, -unpack и -from -to

Из минусов
- Командные файлы надо переделывать :)

- - - Добавлено - - -

В целом - это с прицелом на BRU и ODS-2 :)

Выловил ещё одну ошибку, связанную с переполнением. Ох не зря DEC для RT-11 сделала максимальный размер тома 65535 блоков, а не 65536 (32 Mб) - вот тут у меня переполнение и вылезло :)

Вроде с включением в общий механизм ФС BRU и ODS-2 сделал всё, теперь, собственно, реализовать надо. Начну, скорее всего с BRU для лент :) Так как когда давно делал извлечение файлов с лент BRU. На СМ-2П. На Фортране :)

Развлекаясь и тестируя разное.. Пришло в голову, что конвертирование образов с одного устройства в другое можно малость упростить. Сделать указание размера (если выходное устройство переменного размера) необязательным. Не указали - берём размер входного образа и вуаля :)

Было

ImageUtilsX -from ifile rx01 -to ofile rx01n (типа - хочу снять интерлейсинг)
или
ImageUtilsX -size 1M -from ifile rx01 -to ofile disk (по смыслу тоже, но для выходного размер указан)

Стало

ImageUtilsX -from ifile rx01 -to ofile disk

Но, конечно, никто не запрещает указания размера :)