Re: [devel] Q: rpmvercmp

["Kirill A. Shutemov" <kirill@shutemov.name>]

[-- Attachment #1: Type: text/plain, Size: 2431 bytes --]

On Mon, Jan 12, 2009 at 05:21:06PM +0200, Kirill A. Shutemov wrote:
> Словил забавный баг(фичу?) в rpmvercmp. Есть две версии "1.a" и "1a". 
> rpmvercmp считает их равными! 
> 
> rpm сравнивает эти версии следующим образом (упрощено):
> 
> 1. Отбрасывает из начала обоих строк символы не являющиеся буквами или
>    цифрами (!isalnum) в начале строк. В данном случае в обоих строках 
>    таких символов нет.
> 2. Выбирает из обоих строк максимальное количество количество цифр и
>    сравнивает, как числа. 1 == 1, т.е. на данном этапе версии равны.
> 3. Далее сравнивает оставшиеся части версий: ".a" и "a".
> 4. Тут цикл начинается с начала. Опять отбрасываются все символы не
>    являющиеся буквами и цифрами. Остаётся "a" и "a". Они, очевидно,
>    являются равными.
> 
> Я считаю такое поведение ошибочным. Отбрасывать символы не являющиеся
> буквами или цифрами следует только, если в обоих строках они есть:
> 
> diff --git a/rpmdb/rpmvercmp.c b/rpmdb/rpmvercmp.c
> index 5ac54dd..1539ae5 100644
> --- a/rpmdb/rpmvercmp.c
> +++ b/rpmdb/rpmvercmp.c
> @@ -34,8 +34,10 @@ int rpmvercmp(const char * a, const char * b)
>  
>      /* loop through each version segment of str1 and str2 and compare them */
>      while (*one && *two) {
> -	while (*one && !xisalnum(*one)) one++;
> -	while (*two && !xisalnum(*two)) two++;
> +	if ((*one && !xisalnum(*one)) && (*two && !xisalnum(*two))) {
> +	    while (*one && !xisalnum(*one)) one++;
> +	    while (*two && !xisalnum(*two)) two++;
> +	}
>  
>  	if ( !*one && !*two )
>  		return 0;
> 
> Как вы считаете?

Поскольку тут меня игнорируют, завёл баг #18493. Может так что-нибудь
ответят.

-- 
Regards,  Kirill A. Shutemov
 + Belarus, Minsk
 + ALT Linux Team, http://www.altlinux.org/

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 197 bytes --]