NA> Я имею устойчивый репродакшен баг, на голдеде, построенном из гитхаб NA> мастер.
Вот такой простой фикс чинит проблему, с которой я сталкнулся, когда перцы делают Ориджин длиннее 160 символов.
-+- a/golded3/geline.cpp +++ b/golded3/geline.cpp @@ -1740,7 +1740,7 @@ void ScanKludges(GMsg* msg, int getvalue) strxcpy(msg->origin, line->txt.c_str()+11, sizeof(msg->origin)); if(nextor) // Get the next line too { - strcat(msg->origin, line->next->txt.c_str()); + strxcat(msg->origin, line->next->txt.c_str(), sizeof(msg->origin)); line->next->color = C_READO; line->next->type |= GLINE_ORIG; // Mark next line as Origin too }
P.S. Бонусом расскажу прикол заодно. Здесь sizeof(msg->origin) можно написать без скобок, потому sizeof не функция, а оператор. Если sizeof с именем переменной, то скобки можно не ставить, а если с типом надо писать. Казалось бы, кого это волнует? А вот, у меня на работе такие умники есть, на код-ревью мне такие комментарии пишут, просто в остальном сложно по коду докопаться. :-)
NA> Program received signal SIGABRT, Aborted. NA> gdb) bt NA> #0 0x00007ffff7099c37 in __GI_raise (sig=sig@entry=6) at NA> ../nptl/sysdeps/unix/sysv/linux/raise.c:56 #1 0x00007ffff709d028 in NA> __GI_abort () at abort.c:89 #2 0x00007ffff70d62a4 in __libc_message NA> (do_abort=do_abort@entry=2, fmt=fmt@entry=0x7ffff71e5db0 ";*** %s ***: NA> %s terminated\n") at ../sysdeps/posix/libc_fatal.c:175 #3 NA> 0x00007ffff717187c in __GI___fortify_fail (msg=<optimized out>, NA> msg@entry=0x7ffff71e5d47 "buffer overflow detected") at NA> fortify_fail.c:38 #4 0x00007ffff7170750 in __GI___chk_fail () at NA> chk_fail.c:28 #5 0x00007ffff716f8eb in __strcat_chk ( NA> dest=dest@entry=0xd20488 "\351 \323\324\301\314\311 NA> \304\322\301\324\330\323\321 \351\314\330\321 NA> \355\325\322\317\315\305\303 \323\317 \332\315\305\305\315 NA> \347\317\322\331\316\331\336\305\315 \311 \317\324\322\325\302\311\314 NA> \355\325\322\317\315\305\303 \317\304\316\325 \307\317\314\317\327\325 NA> \372\315\305\300. \356\317 \327\331\322\317\323\314\317 \3257 #6 NA> 0x000000000047dad6 in strcat (__src=<optimized out>, NA> __dest=0xd20488 "\351 \323\324\301\314\311 NA> \304\322\301\324\330\323\321 \351\314\330\321 NA> \355\325\322\317\315\305\303 \323\317 \332\315\305\305\315 NA> \347\317\322\331\316\331\336\305\315 \311 \317\324\322\325\302\311\314 NA> \355\325\322\317\315\305\303 \317\304\316\325 \307\317\314\317\327\325 NA> \372\315\305\300. \356\317 \327\331\322\317\323\314\317 \325 \324\3172 NA> #7 ScanKludges (msg=msg@entry=0xd1ebec, getvalue=getvalue@entry=1) at NA> /home/fido/src/golded-plus/golded3/geline.cpp:1743 #8 NA> 0x000000000047fe80 in MakeLineIndex (msg=0xd1ebec, margin=141, NA> getvalue=true, header_recode=<optimized out>) at NA> /home/fido/src/golded-plus/golded3/geline.cpp:3071 #9 NA> 0x00000000004817e9 in Area::LoadMsg (this=this@entry=0xbab570, NA> msg=msg@entry=0xd1ebec, msgno=8388, margin=margin@entry=141, NA> mode=mode@entry=0) at NA> /home/fido/src/golded-plus/golded3/gelmsg.cpp:134 #10 NA> 0x000000000046fd9b in FindString (msg=0xd1ebec, prompt=<optimized NA> out>, what=<optimized out>) at NA> /home/fido/src/golded-plus/golded3/gmarea.h:1214 #11 NA> 0x00000000004701d1 in FindAll (msg=msg@entry=0xd1ebec, NA> topline=@0x7ff8f4: 0, keyok=@0x7ff8f0: 0) at NA> /home/fido/src/golded-plus/golded3/gmarea.h:985 #12 0x00000000004a7520 NA> in Reader () at /home/fido/src/golded-plus/golded3/geread.cpp:872 #13 NA> 0x000000000040999e in main (argc=<optimized out>, argv=<optimized NA> out>) at /home/fido/src/golded-plus/golded3/gemain.cpp:54
Best Regards, Nil --- GoldED+/LNX 1.1.5 * Origin: Linux 2.6.32-042stab145.3 (2:5015/46)
