[devel] [PATCH for apt] Fix crash when removing 3rd-party packages with some tags missing
Aleksei Nikiforov
darktemplar на altlinux.org
Пт Янв 31 18:48:58 MSK 2020
31.01.2020 17:53, Ivan Zakharyaschev пишет:
> Hello!
>
> Спасибо! А есть примеры пакетов, с которыми можно проверить?
>
Да. google-chrome, rpm с официального сайта.
Если не поставить google-chrome-preinstall, то при его установке пакет
может попытаться через "rpm --import" импортировать ключи, и в
зависимости от условий, которые я пока что не выяснял, либо будет
сообщение об ошибке о том, что /var/lib/rpm/.rpm.lock занят, либо при
импорте двух таких ключей в системе создадутся два пакета с именем
gpg-pubkey.
В принципе, для воспроизведения проблемы не обязательно даже скачивать и
ставить google-chrome. Достаточно сгенерировать 2 разных gpg ключа (или
взять существующие), экспортировать их публичные части через 'gpg
--export -a $KEYID' в файл и импортировать полученные файлы в rpm через
'rpm --import $filename' от рута.
После этого apt начнёт ругаться на дублированные пакеты gpg-pubkey,
содержащие импортированные ключи, а при попытке их удалить - будет падать.
Данный патч исправляет падение в таком случае.
Второй патч, который я прислал чуть позже, исправляет другую проблему,
связанную с данной ситуацией.
Я добавил конфигурацию в файл /etc/apt/apt.conf.d/chrome.conf следующего
вида:
RPM
{
Allow-Duplicated {
"^gpg-pubkey";
};
};
После применения этого патча apt не падает при удалении пакетов
gpg-pubkey, а указанный конфиг убирает предупреждение об их
дублировании. Однако, команда apt-get remove gpg-pubkey выдаёт
сообщение, что в системе нет пакетов с именем gpg-pubkey, хотя rpm -q
gpg-pubkey говорит о наличии нескольких таких пакетов. Второй патч
соответственно исправляет эту ситуацию, приводя работу с такими пакетами
к такому же виду, в каком apt обрабатывает другие дублированные пакеты
из репозитория, например, kernel-image-un-def, т.е. apt выдаёт все
имеющиеся версии и предлагает пользователю указать нужные.
> On Fri, 31 Jan 2020, Aleksei Nikiforov wrote:
>
>> ---
>> apt/apt-pkg/rpm/rpmpm.cc | 7 ++++++-
>> 1 file changed, 6 insertions(+), 1 deletion(-)
>>
>> diff --git a/apt/apt-pkg/rpm/rpmpm.cc b/apt/apt-pkg/rpm/rpmpm.cc
>> index e00630f..4bca9f4 100644
>> --- a/apt/apt-pkg/rpm/rpmpm.cc
>> +++ b/apt/apt-pkg/rpm/rpmpm.cc
>> @@ -81,7 +81,12 @@ std::string rpm_name_conversion(const pkgCache::PkgIterator &Pkg)
>> #if RPM_VERSION >= 0x040202
>> // This is needed for removal to work on multilib packages, but old
>> // rpm versions don't support name.arch in RPMDBI_LABEL, oh well...
>> - Name = Name + "." + Pkg.CurrentVer().Arch();
>> + // Note: some 3rd-party packages may still miss arch, so only use it
>> + // when it's present
>> + if (Pkg.CurrentVer().Arch() != nullptr)
>> + {
>> + Name = Name + "." + Pkg.CurrentVer().Arch();
>> + }
>> #endif
>>
>> return Name;
>> --
>> 2.24.1
>
>
>
> _______________________________________________
> Devel mailing list
> Devel на lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel
>
Подробная информация о списке рассылки Devel