21684 2009-09-23 14:39:36 +0400 ошибка в strfmon() 2010-05-27 04:46:02 +0400 1 1 4 Development Sisyphus glibc-core unstable all Linux CLOSED FIXED http://sourceware.org/bugzilla/show_bug.cgi?id=10600 P3 normal --- 1 stalker glebfm damir erthad glebfm ldv mike php-coder qa-sisyphus oldest_to_newest 99922 0 stalker 2009-09-23 14:39:36 +0400 На 4.0 успешно срабатывает тесткейз (см ниже) на ошибку по поводу strfmon в glibc. На 32 битных хостах получается просто сегфолт..а вот на 64 битном куда серьёзнее - съедается вся доступная память. Аналогичные симптомы в 4.1 и в p5. == #include <stdio.h> #include <monetary.h> void main(void) { double p = 1; char str[20]; strfmon(str, sizeof(str), "%.1073741821i", p); } или оно же на php php -r 'money_format("%.1073741821i",1);' 99929 1 ldv 2009-09-23 15:10:04 +0400 Апстрим не торопится это исправлять. История может предствлять реальную угрозу только в том случае, когда строка формата, передаваемая функции strfmon, является непроверенным вводом. Последнее плохо в любом случае. 99946 2 stalker 2009-09-23 16:10:02 +0400 То что php не показатель понятно.Но тем не менее сегодня 2 ovz-контейнера ушли в даун... там как раз используется money_format. Отключить использование функции не могу..ибо оно нужно. 99978 3 3931 damir 2009-09-23 19:10:54 +0400 Created attachment 3931 Примерный патч для glibc-2.3.5 Вот с таким патчем у меня на glibc-2.3.5 (своя собственная embedded-версия) не падает. Хотя конечно, правильнее было бы патчить __printf_fp, чтобы он не писал куда не надо. 109486 4 ldv 2010-05-27 04:46:02 +0400 По крайней мере в glibc >= 2.10.1-alt2 это уже исправлено. В 5.1/p5 сейчас находится glibc-2.10.1-alt7. 3931 2009-09-23 19:10:54 +0400 2009-09-23 19:10:54 +0400 Примерный патч для glibc-2.3.5 strfmon_l.patch text/plain 972 damir SW5kZXg6IHN0cmZtb25fbC5jCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIHN0cmZtb25fbC5jCShyZXZpc2lvbiA4 MCkKKysrIHN0cmZtb25fbC5jCSh3b3JraW5nIGNvcHkpCkBAIC0yMzMsNyArMjMzLDcgQEAKIAog CSAgLyogSWYgd2UgZG9uJ3QgaGF2ZSBlbm91Z2ggcm9vbSBmb3IgdGhlIGRlbWFuZGVkIHdpZHRo IHdlCiAJICAgICBjYW4gc3RvcCBub3cgYW5kIHJldHVybiBhbiBlcnJvci4gICovCi0JICBpZiAo ZGVzdCArIHdpZHRoID49IHMgKyBtYXhzaXplKQorCSAgaWYgKHdpZHRoIDwgMCB8fCBkZXN0ICsg d2lkdGggPj0gcyArIG1heHNpemUpCiAJICAgIHsKIAkgICAgICBfX3NldF9lcnJubyAoRTJCSUcp OwogCSAgICAgIHJldHVybiAtMTsKQEAgLTI1NSw2ICsyNTUsMTEgQEAKIAkgICAgICBsZWZ0X3By ZWMgKj0gMTA7CiAJICAgICAgbGVmdF9wcmVjICs9IHRvX2RpZ2l0ICgqZm10KTsKIAkgICAgfQor CSAgaWYgKGxlZnRfcHJlYyA8IDAgfHwgZGVzdCArIGxlZnRfcHJlYyA+PSBzICsgbWF4c2l6ZSkK KwkgICAgeworCSAgICAgIF9fc2V0X2Vycm5vIChFMkJJRyk7CisJICAgICAgcmV0dXJuIC0xOwor CSAgICB9CiAJfQogCiAgICAgICAvKiBSZWNvZ25pemUgcmlnaHQgcHJlY2lzaW9uLiAgKi8KQEAg LTI3Miw2ICsyNzcsMTEgQEAKIAkgICAgICByaWdodF9wcmVjICo9IDEwOwogCSAgICAgIHJpZ2h0 X3ByZWMgKz0gdG9fZGlnaXQgKCpmbXQpOwogCSAgICB9CisJICBpZiAocmlnaHRfcHJlYyA8IDAg fHwgZGVzdCArIGxlZnRfcHJlYyArIHJpZ2h0X3ByZWMgPj0gcyArIG1heHNpemUgKQorCSAgICB7 CisJICAgICAgX19zZXRfZXJybm8gKEUyQklHKTsKKwkgICAgICByZXR1cm4gLTE7CisJICAgIH0K IAl9CiAKICAgICAgIC8qIEhhbmRsZSBtb2RpZmllci4gIFRoaXMgaXMgYW4gZXh0ZW5zaW9uLiAg Ki8K