[devel] Q: rpmvercmp

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

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

Словил забавный баг(фичу?) в 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;

Как вы считаете?

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

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