ALT Linux Team development discussions
 help / color / mirror / Atom feed
* [devel] Q: rpmvercmp
@ 2009-01-12 15:21 Kirill A. Shutemov
  2009-01-12 20:03 ` Kirill A. Shutemov
  2009-01-12 21:12 ` Dmitry V. Levin
  0 siblings, 2 replies; 16+ messages in thread
From: Kirill A. Shutemov @ 2009-01-12 15:21 UTC (permalink / raw)
  To: devel

[-- 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 --]

^ permalink raw reply	[flat|nested] 16+ messages in thread

end of thread, other threads:[~2009-02-17 23:30 UTC | newest]

Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-01-12 15:21 [devel] Q: rpmvercmp Kirill A. Shutemov
2009-01-12 20:03 ` Kirill A. Shutemov
2009-01-12 21:12 ` Dmitry V. Levin
2009-01-12 21:19   ` Led
2009-01-13  6:25   ` Kirill A. Shutemov
2009-01-13 21:37     ` Dmitry V. Levin
2009-01-14  8:06       ` Kirill A. Shutemov
2009-01-19 11:14         ` Kirill A. Shutemov
2009-01-27  1:32           ` Dmitry V. Levin
2009-01-27  3:18             ` Alexey Morozov
2009-01-27  8:21             ` Led
2009-01-27 10:18               ` Ivan Fedorov
2009-02-16 23:38               ` Dmitry V. Levin
2009-02-17  2:30                 ` Ivan Fedorov
2009-02-17 10:58                 ` Led
2009-02-17 23:30                   ` Kirill A. Shutemov

ALT Linux Team development discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \
		devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
	public-inbox-index devel

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.devel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git