8514 2005-11-18 01:20:15 +0300 работа в 64 битном режиме 2005-11-23 01:17:43 +0300 1 1 4 Development Sisyphus wm-select unstable all Linux CLOSED FIXED P2 normal --- 1 damir nobody qa-sisyphus oldest_to_newest 33166 0 damir 2005-11-18 01:20:17 +0300 Программа не работает в 64 битном режиме, потому что в функции FillBox структура копируется в строку, в которой недостаточно места для всей структуры. В 64 битном режиме структура wm_entry имеет размер 32 байта, что превышает размер строки, в которую эта строка копируется в функции FillBox. 33167 1 1252 damir 2005-11-18 01:22:43 +0300 Created attachment 1252 Proposed patch to fix string overflow Прилагаю патч, в котором в строку копируется не вся структура, а только ее указатель - это 4 или 8 байт. Место под структуру аллоцируется malloc-ом и освобождается через free после создания кнопки для wm. 33241 2 ldv 2005-11-21 03:36:17 +0300 Fixed in wm-select-0.3 (rewritten FillBox() in more portable way: struct dirent array should not be used for custom purposes). 1252 2005-11-18 01:22:43 +0300 2005-11-18 01:22:43 +0300 Proposed patch to fix string overflow wm-select-0.2-alt-fix64bit.patch text/plain 1772 damir ZGlmZiAtcnUgd20tc2VsZWN0LTAuMi5vcmlnL3dtLXNlbGVjdC5jIHdtLXNlbGVjdC0wLjIvd20t c2VsZWN0LmMKLS0tIHdtLXNlbGVjdC0wLjIub3JpZy93bS1zZWxlY3QuYwkyMDA1LTExLTE4IDAw OjM0OjQwICswMzAwCisrKyB3bS1zZWxlY3QtMC4yL3dtLXNlbGVjdC5jCTIwMDUtMTEtMTggMDE6 MTA6MTEgKzAzMDAKQEAgLTM1NSwzNyArMzU1LDM3IEBACiAKIAlmb3IgKGkgPSAwOyBpIDwgbjsg KytpKQogCXsKLQkJd21fZW50cnkgZW50cnk7CisJCXdtX2VudHJ5KiBlbnRyeSA9IG1hbGxvYyhz aXplb2Yod21fZW50cnkpKTsKIAkJY2hhciAgICpmbmFtZTsKIAogCQlpZiAoYXNwcmludGYgKCZm bmFtZSwgIiVzLyVzIiwgV01fRElSLCBuYW1lbGlzdFtpXS0+ZF9uYW1lKSA8CiAJCSAgICAwKQog CQkJZXJyb3IgKEVYSVRfRkFJTFVSRSwgZXJybm8sICJhc3ByaW50ZiIpOwotCQlSZWFkV01Db25m aWcgKGZuYW1lLCAmZW50cnkpOwotCQlpZiAoIWVudHJ5Lm5hbWUgfHwgIWVudHJ5Lmljb24gfHwg IWVudHJ5LmV4ZWMpCisJCVJlYWRXTUNvbmZpZyAoZm5hbWUsIGVudHJ5KTsKKwkJaWYgKCFlbnRy eS0+bmFtZSB8fCAhZW50cnktPmljb24gfHwgIWVudHJ5LT5leGVjKQogCQl7CiAJCQllcnJvciAo RVhJVF9TVUNDRVNTLCAwLCAiJXM6IGNvbmZpZyBpbmNvbXBsZXRlIiwKIAkJCSAgICAgICBmbmFt ZSk7CiAJCQlmcmVlIChmbmFtZSk7Ci0JCQlpZiAoZW50cnkubmFtZSkKLQkJCQlmcmVlIChlbnRy eS5uYW1lKTsKLQkJCWlmIChlbnRyeS5pY29uKQotCQkJCWZyZWUgKGVudHJ5Lmljb24pOwotCQkJ aWYgKGVudHJ5LmV4ZWMpCi0JCQkJZnJlZSAoZW50cnkuZXhlYyk7Ci0JCQllbnRyeS52YWxpZCA9 IDA7CisJCQlpZiAoZW50cnktPm5hbWUpCisJCQkJZnJlZSAoZW50cnktPm5hbWUpOworCQkJaWYg KGVudHJ5LT5pY29uKQorCQkJCWZyZWUgKGVudHJ5LT5pY29uKTsKKwkJCWlmIChlbnRyeS0+ZXhl YykKKwkJCQlmcmVlIChlbnRyeS0+ZXhlYyk7CisJCQllbnRyeS0+dmFsaWQgPSAwOwogCQkJbWVt Y3B5IChuYW1lbGlzdFtpXS0+ZF9uYW1lLCAoY29uc3QgY2hhciAqKSAmZW50cnksCiAJCQkJc2l6 ZW9mIChlbnRyeSkpOwogCQkJY29udGludWU7CiAJCX0KIAkJZnJlZSAoZm5hbWUpOwotCQllbnRy eS52YWxpZCA9IDE7CisJCWVudHJ5LT52YWxpZCA9IDE7CiAJCW1lbWNweSAobmFtZWxpc3RbaV0t PmRfbmFtZSwgKGNvbnN0IGNoYXIgKikgJmVudHJ5LAogCQkJc2l6ZW9mIChlbnRyeSkpOwogCQkr K3RvdGFsOwogCiAJCWlmICghcHJlZmVycmVkKQotCQkJcHJlZmVycmVkID0gZW50cnkuZXhlYzsK KwkJCXByZWZlcnJlZCA9IGVudHJ5LT5leGVjOwogCX0KIAogCWlmICh0b3RhbCA8PSA2KQpAQCAt NDAwLDcgKzQwMCw3IEBACiAJLy8gZXJyb3IgKEVYSVRfU1VDQ0VTUywgMCwgInRvdGFsPSVkLCB3 cmFwX2xpbWl0PSVkIiwgdG90YWwsIHdyYXBfbGltaXQpOwogCWZvciAoaSA9IDA7IGkgPCBuOyAr K2kpCiAJewotCQl3bV9lbnRyeSAqZW50cnkgPSAod21fZW50cnkgKikgKG5hbWVsaXN0W2ldLT5k X25hbWUpOworCQl3bV9lbnRyeSAqZW50cnkgPSAqKCh3bV9lbnRyeSAqKikgKG5hbWVsaXN0W2ld LT5kX25hbWUpKTsKIAogCQlpZiAoIWVudHJ5LT52YWxpZCkKIAkJCWNvbnRpbnVlOwpAQCAtNDE3 LDYgKzQxNyw3IEBACiAJCQloYm94ID0gZ3RrX2hib3hfbmV3ICgxLCAwKTsKIAkJCXdyYXBfaSA9 IDA7CiAJCX0KKwkJZnJlZShlbnRyeSk7CiAJfQogCWZyZWUgKG5hbWVsaXN0KTsKIAlpZiAod3Jh cF9pKQo=