From 9bb9d3abd85464516570b935467bb97b9a96d95b Mon Sep 17 00:00:00 2001 From: Ianos Gnatiuc Date: Mon, 10 Jul 2006 17:30:59 +0000 Subject: [PATCH] fixed buffer overflow bug. golded+ can crash in message list if size of name or subject is too big. --- docs/notework.rus | 2 ++ docs/notework.txt | 2 ++ golded3/gemlst.cpp | 7 ++++--- 3 files changed, 8 insertions(+), 3 deletions(-) diff --git a/docs/notework.rus b/docs/notework.rus index 1e7e842..e44bffa 100644 --- a/docs/notework.rus +++ b/docs/notework.rus @@ -10,6 +10,8 @@ _____________________________________________________________________________ Заметки для GoldED+ 1.1.5, /snapshot/ _____________________________________________________________________________ +- Golded+ мог упасть в списке сообщений, если размер имени или темы слишком + большие. - Исправлен меню удаления неотправленных сообщений, когда удалялось в корзину. - Исправлен подсчет времени сколько голдед простаивал. - Не работало ^буква из командной строки. diff --git a/docs/notework.txt b/docs/notework.txt index 05adb0c..bc7b5be 100644 --- a/docs/notework.txt +++ b/docs/notework.txt @@ -10,6 +10,8 @@ ______________________________________________________________________ Notes for GoldED+ 1.1.5, /snapshot/ ______________________________________________________________________ +- Golded+ can crash in message list if size of name or subject is too big. + - Fixed delete menu dialog, when unsent message is to be deleted to recyclebin. - Fixed idle time counting. diff --git a/golded3/gemlst.cpp b/golded3/gemlst.cpp index fc30e6d..4042104 100644 --- a/golded3/gemlst.cpp +++ b/golded3/gemlst.cpp @@ -221,9 +221,10 @@ void GMsgList::ReadMlst(int n) { ml->written = msg.written; ml->arrived = msg.arrived; ml->received = msg.received; - strcpy(ml->by, msg.By()); - strcpy(ml->to, msg.To()); - strcpy(ml->re, msg.re); + + strxcpy(ml->by, msg.By(), ARRAYSIZE(ml->by)); + strxcpy(ml->to, msg.To(), ARRAYSIZE(ml->to)); + strxcpy(ml->re, msg.re, ARRAYSIZE(ml->re)); { Addr zero; ml->colorby = GetColorName(ml->by, msg.orig, DEFATTR);