12628 2007-08-25 14:08:37 +0400 segmentation fault netstat -I eth0 2008-12-03 12:55:04 +0300 1 1 4 Development Sisyphus net-tools unstable all Linux CLOSED FIXED patch P2 normal --- 1 viy placeholder glebfm icesik ldv php-coder placeholder vt qa-sisyphus oldest_to_newest 54476 0 viy 2007-08-25 14:08:37 +0400 $ rpm -qf /bin/netstat net-tools-1.60-alt13 igor@breeze ~ $ sudo netstat -I eth0 zsh: segmentation fault sudo netstat -I eth0 igor@breeze ~ $ uname -a Linux breeze.localdomain 2.6.18-ovz-smp-alt14 #1 SMP Wed May 2 15:41:34 MSD 2007 x86_64 GNU/Linux 55378 1 ldv 2007-09-18 12:49:08 +0400 confirmed; who wants to update net-tools patchset from FC? 59260 2 icesik 2007-12-10 04:56:53 +0300 (In reply to comment #1) > confirmed; who wants to update net-tools patchset from FC? i will sync with patches with F and report about this. 74148 3 php-coder 2008-07-18 10:04:36 +0400 (In reply to comment #2) > i will sync with patches with F and report about this. *ping* 74149 4 php-coder 2008-07-18 10:49:42 +0400 Как netstat -I (без аргумента) тоже падает. 74150 5 php-coder 2008-07-18 11:04:02 +0400 Ошибка в этом куске кода (файл netstat.c): 1787 case 'I': 1788 if (optarg && strcmp(optarg, "(null)")) 1789 if (optarg[0] == '=') optarg++; 1790 flag_int_name = strdup(optarg); 1791 flag_int++; 1792 break; Первый if проверяет, чтобы optarg не была NULL, проверяет довольно странно (я бы проверял как otarg != NULL или просто optarg), но идея в этом. Второй if проверяет случай когда опция укзаана как -I=eth0. Если аргумент знак равно, то переходим к следующему аргументу строки. И наконец третья строка пытается сохранить в flag_int_name копию аргумента (eth0). Падает всё в случае, когда optarg == NULL. Судя по отступам можно понять чего автор хотел, вот только он забыл добавить скобок. Он хотел, чтобы строка 1789 и 1790 выполнялись, только если первый if вернул истину. Но получилось, что строка 1790 выполняется всегда, т.к. не принадлежит телу первого if-а. Кстати, в оригинальном коде программы этого куска кода нет. Он привнесен патчами net-tools-1.60-rh-netstat-ifaceopt.patch и net-tools-1.60-rh-netstat-I.patch (кстати, патчи добавляют опцию -I но не патчат man, в итоге а мане этой опции нет, что тоже баг). 74151 6 php-coder 2008-07-18 11:15:58 +0400 А, ну всё понятно. Вот каким должен быть код: 1787 case 'I': 1788 if (optarg != NULL) { 1789 if (optarg[0] == '=') { 1790 optarg++; 1791 } 1792 flag_int_name = strdup(optarg); 1793 } 1794 flag_int++; 1795 break; Обратите внимание, что вызывать netstat -I eth0 -- НЕ правильно. Надо: netstat -I=eth0 Вот валидные опции: - I - --interfaces - I=eth0 - --interfaces=eth0 Вот так с приведенным выше кодом всё будет работать. А на -I eth0 будет выдавать usage. 74153 7 2708 php-coder 2008-07-18 11:40:24 +0400 Created attachment 2708 Updated patch version Для исправления этой баги нужно: - поравить спек (не прикладывать патч net-tools-1.60-rh-netstat-I.patch) - удалить патч net-tools-1.60-rh-netstat-I.patch - заменить патч net-tools-1.60-rh-netstat-ifaceopt.patch на тот, который я сейчас прикладываю После этого всё работает. 81952 8 ldv 2008-11-30 18:56:51 +0300 Вроде бы fixed in 1.60-alt14 2708 2008-07-18 11:40:24 +0400 2008-07-18 11:40:24 +0400 Updated patch version net-tools-1.60-rh-netstat-ifaceopt.patch text/plain 2145 php-coder LS0tIG5ldC10b29scy0xLjYwL25ldHN0YXQuYy5pZmFjZW9wdAkyMDA1LTAzLTAxIDEyOjExOjQz LjY5NTY2MTYzMiArMDEwMAorKysgbmV0LXRvb2xzLTEuNjAvbmV0c3RhdC5jCTIwMDUtMDMtMDEg MTI6MTY6MzcuNjAxOTgxMDgwICswMTAwCkBAIC0xNjU4LDEwICsxNjU4LDExIEBACiB7CiAgICAg ZnByaW50ZihzdGRlcnIsIF8oInVzYWdlOiBuZXRzdGF0IFstdmVlbk5jQ0ZdIFs8QWY+XSAtciAg ICAgICAgIG5ldHN0YXQgey1WfC0tdmVyc2lvbnwtaHwtLWhlbHB9XG4iKSk7CiAgICAgZnByaW50 ZihzdGRlcnIsIF8oIiAgICAgICBuZXRzdGF0IFstdm5OY2Flb2xdIFs8U29ja2V0PiAuLi5dXG4i KSk7Ci0gICAgZnByaW50ZihzdGRlcnIsIF8oIiAgICAgICBuZXRzdGF0IHsgWy12ZWVuTmFjXSAt aVs8SWZhY2U+XSB8IFstY25OZV0gLU0gfCAtcyB9IFtkZWxheV1cblxuIikpOworICAgIGZwcmlu dGYoc3RkZXJyLCBfKCIgICAgICAgbmV0c3RhdCB7IFstdmVlbk5hY10gLUlbPElmYWNlPl0gfCBb LXZlZW5OYWNdIC1pIHwgWy1jbk5lXSAtTSB8IC1zIH0gW2RlbGF5XVxuXG4iKSk7CiAKICAgICBm cHJpbnRmKHN0ZGVyciwgXygiICAgICAgICAtciwgLS1yb3V0ZSAgICAgICAgICAgICAgICBkaXNw bGF5IHJvdXRpbmcgdGFibGVcbiIpKTsKLSAgICBmcHJpbnRmKHN0ZGVyciwgXygiICAgICAgICAt aSwgLS1pbnRlcmZhY2VzPVs8SWZhY2U+XSBkaXNwbGF5IGludGVyZmFjZSB0YWJsZVxuIikpOwor ICAgIGZwcmludGYoc3RkZXJyLCBfKCIgICAgICAgIC1JLCAtLWludGVyZmFjZXM9WzxJZmFjZT5d IGRpc3BsYXkgaW50ZXJmYWNlIHRhYmxlIGZvciA8SWZhY2U+XG4iKSk7CisgICAgZnByaW50Zihz dGRlcnIsIF8oIiAgICAgICAgLWksIC0taW50ZXJmYWNlcyAgICAgICAgICAgZGlzcGxheSBpbnRl cmZhY2UgdGFibGVcbiIpKTsKICAgICBmcHJpbnRmKHN0ZGVyciwgXygiICAgICAgICAtZywgLS1n cm91cHMgICAgICAgICAgICAgICBkaXNwbGF5IG11bHRpY2FzdCBncm91cCBtZW1iZXJzaGlwc1xu IikpOwogICAgIGZwcmludGYoc3RkZXJyLCBfKCIgICAgICAgIC1zLCAtLXN0YXRpc3RpY3MgICAg ICAgICAgIGRpc3BsYXkgbmV0d29ya2luZyBzdGF0aXN0aWNzIChsaWtlIFNOTVApXG4iKSk7CiAj aWYgSEFWRV9GV19NQVNRVUVSQURFCkBAIC0xNzAwLDcgKzE3MDEsOCBAQAogICAgIHsKIAlBRlRS QU5TX09QVFMsCiAJeyJ2ZXJzaW9uIiwgMCwgMCwgJ1YnfSwKLQl7ImludGVyZmFjZXMiLCAyLCAw LCAnaSd9LAorCXsiaW50ZXJmYWNlcyIsIDIsIDAsICdJJ30sCisJeyJpbnRlcmZhY2VzIiwgMCwg MCwgJ2knfSwKIAl7ImhlbHAiLCAwLCAwLCAnaCd9LAogCXsicm91dGUiLCAwLCAwLCAncid9LAog I2lmIEhBVkVfRldfTUFTUVVFUkFERQpAQCAtMTczOCw3ICsxNzQwLDggQEAKICAgICBnZXRyb3V0 ZV9pbml0KCk7CQkvKiBTZXQgdXAgQUYgcm91dGluZyBzdXBwb3J0ICovCiAKICAgICBhZm5hbWVb MF0gPSAnXDAnOwotICAgIHdoaWxlICgoaSA9IGdldG9wdF9sb25nKGFyZ2MsIGFyZ3YsICJNQ0ZB OmFjZGVncGhpOjpuTm9yc3R1VnY/d3hsIiwgbG9uZ29wdHMsICZsb3ApKSAhPSBFT0YpCisKKyAg ICB3aGlsZSAoKGkgPSBnZXRvcHRfbG9uZyhhcmdjLCBhcmd2LCAiTUNGQTphY2RlZ3BoaUk6Om5O b3JzdHVWdj93eGwiLCBsb25nb3B0cywgJmxvcCkpICE9IEVPRikKIAlzd2l0Y2ggKGkpIHsKIAlj YXNlIC0xOgogCSAgICBicmVhazsKQEAgLTE3NzMsOSArMTc3MywxNiBAQAogCWNhc2UgJ3AnOgog CSAgICBmbGFnX3ByZysrOwogCSAgICBicmVhazsKKwljYXNlICdJJzoKKwkgICAgaWYgKG9wdGFy ZyAhPSBOVUxMKSB7CisJICAgICAgICBpZiAob3B0YXJnWzBdID09ICc9JykgeworCSAgICAgICAg ICAgIG9wdGFyZysrOworCSAgICAgICAgfQorCSAgICAgICAgZmxhZ19pbnRfbmFtZSA9IHN0cmR1 cChvcHRhcmcpOworCSAgICB9CisJICAgIGZsYWdfaW50Kys7CisJICAgIGJyZWFrOwogCWNhc2Ug J2knOgotICAgICAgICAgICAgaWYgKG9wdGFyZyAmJiBzdHJjbXAob3B0YXJnLCAiKG51bGwpIikp Ci0gICAgICAgICAgICAgICAgZmxhZ19pbnRfbmFtZSA9IHN0cmR1cChvcHRhcmcpOwogCSAgICBm bGFnX2ludCsrOwogCSAgICBicmVhazsKIAljYXNlICduJzoK