summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorkcwu <kcwu@63ad8ddf-47c3-0310-b6dd-a9e9d9715204>2003-09-07 13:49:13 +0800
committerkcwu <kcwu@63ad8ddf-47c3-0310-b6dd-a9e9d9715204>2003-09-07 13:49:13 +0800
commit64fa801fe4bd90430d601b72376a3eb74d7639fa (patch)
tree9d26964ba0179c7fb45cd377dc854d84124a5b8a
parent4090fbd716e44c899e25e4bfa3d1376620051b62 (diff)
downloadpttbbs-64fa801fe4bd90430d601b72376a3eb74d7639fa.tar
pttbbs-64fa801fe4bd90430d601b72376a3eb74d7639fa.tar.gz
pttbbs-64fa801fe4bd90430d601b72376a3eb74d7639fa.tar.bz2
pttbbs-64fa801fe4bd90430d601b72376a3eb74d7639fa.tar.lz
pttbbs-64fa801fe4bd90430d601b72376a3eb74d7639fa.tar.xz
pttbbs-64fa801fe4bd90430d601b72376a3eb74d7639fa.tar.zst
pttbbs-64fa801fe4bd90430d601b72376a3eb74d7639fa.zip
code cleanup
git-svn-id: http://opensvn.csie.org/pttbbs/trunk/pttbbs@1168 63ad8ddf-47c3-0310-b6dd-a9e9d9715204
-rw-r--r--mbbsd/bbs.c2
-rw-r--r--mbbsd/chat.c124
-rw-r--r--mbbsd/mail.c2
-rw-r--r--mbbsd/more.c17
-rw-r--r--mbbsd/talk.c19
5 files changed, 54 insertions, 110 deletions
diff --git a/mbbsd/bbs.c b/mbbsd/bbs.c
index defd0061..bb7d7f9a 100644
--- a/mbbsd/bbs.c
+++ b/mbbsd/bbs.c
@@ -1436,7 +1436,7 @@ del_post(int ent, fileheader_t * fhdr, char *direct)
return DONOTHING;
getdata(1, 0, msg_del_ny, genbuf, 3, LCECHO);
- if (genbuf[0] == 'y' || genbuf[0] == 'Y') {
+ if (genbuf[0] == 'y') {
strlcpy(currfile, fhdr->filename, sizeof(currfile));
if (!delete_file(direct, sizeof(fileheader_t), ent, cmpfilename)) {
if (currmode & MODE_SELECT) {
diff --git a/mbbsd/chat.c b/mbbsd/chat.c
index 5493608f..d7ea2f53 100644
--- a/mbbsd/chat.c
+++ b/mbbsd/chat.c
@@ -1,7 +1,8 @@
/* $Id$ */
#include "bbs.h"
-static int chatline, stop_line;
+#define STOP_LINE (t_lines-3)
+static int chatline;
static FILE *flog;
static void
printchatline(char *str)
@@ -9,11 +10,11 @@ printchatline(char *str)
move(chatline, 0);
if (*str == '>' && !PERM_HIDE(currutmp))
return;
- else if (chatline < stop_line - 1)
+ else if (chatline < STOP_LINE - 1)
chatline++;
else {
- region_scroll_up(2, stop_line - 2);
- move(stop_line - 2, 0);
+ region_scroll_up(2, STOP_LINE - 2);
+ move(STOP_LINE - 2, 0);
}
outs(str);
outc('\n');
@@ -26,7 +27,7 @@ printchatline(char *str)
static void
chat_clear()
{
- for (chatline = 2; chatline < stop_line; chatline++) {
+ for (chatline = 2; chatline < STOP_LINE; chatline++) {
move(chatline, 0);
clrtoeol();
}
@@ -61,9 +62,8 @@ static char chatroom[IDLEN];/* Chat-Room Name */
static int
chat_recv(int fd, char *chatid)
{
- static char buf[512];
+ static char buf[128];
static int bufstart = 0;
- char genbuf[200];
int c, len;
char *bptr;
@@ -94,9 +94,8 @@ chat_recv(int fd, char *chatid)
case 't':
move(0, 0);
clrtoeol();
- snprintf(genbuf, sizeof(genbuf), "談天室 [%s]", chatroom);
- prints("\033[1;37;46m %-21s \033[45m 話題:%-48s\033[m",
- genbuf, bptr + 2);
+ prints("\033[1;37;46m 談天室 [%-12s] \033[45m 話題:%-48s\033[m",
+ chatroom, bptr + 2);
}
} else
printchatline(bptr);
@@ -106,8 +105,7 @@ chat_recv(int fd, char *chatid)
}
if (c > 0) {
- strlcpy(genbuf, bptr, sizeof(genbuf));
- strlcpy(buf, genbuf, sizeof(buf));
+ memmove(buf, bptr, sizeof(buf)-(bptr-buf));
bufstart = len - 1;
} else
bufstart = 0;
@@ -124,7 +122,7 @@ printuserent(userinfo_t * uentp)
if (!uentp) {
if (cnt)
printchatline(uline);
- bzero(uline, 80);
+ bzero(uline, sizeof(uline));
cnt = 0;
return 0;
}
@@ -309,8 +307,6 @@ t_chat()
int newmail;
int chatting = YEA;
char fpath[80];
- char genbuf[200];
- char roomtype;
outs(" 驅車前往 請梢候........ ");
if (!(h = gethostbyname("localhost"))) {
@@ -325,40 +321,30 @@ t_chat()
memcpy(&sin.sin_addr, h->h_addr, h->h_length);
sin.sin_port = htons(NEW_CHATPORT);
cfd = socket(sin.sin_family, SOCK_STREAM, 0);
- if (!(connect(cfd, (struct sockaddr *) & sin, sizeof sin)))
- roomtype = 1;
- else {
- sin.sin_port = CHATPORT;
- cfd = socket(sin.sin_family, SOCK_STREAM, 0);
- if (!(connect(cfd, (struct sockaddr *) & sin, sizeof sin)))
- roomtype = 2;
- else {
- outs("\n "
- "哇! 沒人在那邊耶...要有那地方的人先去開門啦!...");
- system("bin/xchatd");
- pressanykey();
- return -1;
- }
+ if (connect(cfd, (struct sockaddr *) & sin, sizeof sin) != 0) {
+ outs("\n "
+ "哇! 沒人在那邊耶...要有那地方的人先去開門啦!...");
+ system("bin/xchatd");
+ pressanykey();
+ close(cfd);
+ return -1;
}
while (1) {
- getdata(b_lines - 1, 0, "請輸入聊天代號:", inbuf, 9, DOECHO);
- snprintf(chatid, sizeof(chatid),
- "%s", (inbuf[0] ? inbuf : cuser.userid));
+ getdata(b_lines - 1, 0, "請輸入聊天代號:", chatid, 9, DOECHO);
+ if(!chatid[0])
+ strlcpy(chatid, cuser.userid, sizeof(chatid));
chatid[8] = '\0';
/*
- * 舊格式: /! 使用者編號 使用者等級 UserID ChatID 新格式: /!
- * UserID ChatID Password
+ * 新格式: /! UserID ChatID Password
*/
- if (roomtype == 1)
- snprintf(inbuf, sizeof(inbuf), "/! %s %s %s",
- cuser.userid, chatid, cuser.passwd);
- else
- snprintf(inbuf, sizeof(inbuf), "/! %d %d %s %s",
- usernum, cuser.userlevel, cuser.userid, chatid);
+ snprintf(inbuf, sizeof(inbuf), "/! %s %s %s",
+ cuser.userid, chatid, cuser.passwd);
chat_send(cfd, inbuf);
- if (recv(cfd, inbuf, 3, 0) != 3)
+ if (recv(cfd, inbuf, 3, 0) != 3) {
+ close(cfd);
return 0;
+ }
if (!strcmp(inbuf, CHAT_LOGIN_OK))
break;
else if (!strcmp(inbuf, CHAT_LOGIN_EXISTS))
@@ -378,7 +364,7 @@ t_chat()
newmail = currchar = 0;
cmdpos = -1;
- memset(lastcmd, 0, MAXLASTCMD * 80);
+ memset(lastcmd, 0, sizeof(lastcmd));
setutmpmode(CHATING);
currutmp->in_chat = YEA;
@@ -386,15 +372,13 @@ t_chat()
clear();
chatline = 2;
- strlcpy(inbuf, chatid, sizeof(inbuf));
- stop_line = t_lines - 3;
- move(stop_line, 0);
+ move(STOP_LINE, 0);
outs(msg_seperator);
move(1, 0);
outs(msg_seperator);
print_chatid(chatid);
- memset(inbuf, 0, 80);
+ memset(inbuf, 0, sizeof(inbuf));
sethomepath(fpath, cuser.userid);
strlcpy(fpath, tempnam(fpath, "chat_"), sizeof(fpath));
@@ -435,9 +419,7 @@ t_chat()
chatting = chat_send(cfd, "/b");
break;
}
- continue;
- }
- if (isprint2(ch)) {
+ } else if (isprint2(ch)) {
if (currchar < 68) {
if (inbuf[currchar]) { /* insert */
int i;
@@ -452,9 +434,7 @@ t_chat()
move(b_lines - 1, currchar + chatid_len);
outs(&inbuf[currchar++]);
}
- continue;
- }
- if (ch == '\n' || ch == '\r') {
+ } else if (ch == '\n' || ch == '\r') {
if (*inbuf) {
chatting = chat_cmd(inbuf, cfd);
if (chatting == 0)
@@ -473,9 +453,7 @@ t_chat()
}
print_chatid(chatid);
move(b_lines - 1, chatid_len);
- continue;
- }
- if (ch == Ctrl('H') || ch == '\177') {
+ } else if (ch == Ctrl('H') || ch == '\177') {
if (currchar) {
currchar--;
inbuf[69] = '\0';
@@ -484,20 +462,15 @@ t_chat()
clrtoeol();
outs(&inbuf[currchar]);
}
- continue;
- }
- if (ch == Ctrl('Z') || ch == Ctrl('Y')) {
+ } else if (ch == Ctrl('Z') || ch == Ctrl('Y')) {
inbuf[0] = '\0';
currchar = 0;
print_chatid(chatid);
move(b_lines - 1, chatid_len);
- continue;
- }
- if (ch == Ctrl('C')) {
+ } else if (ch == Ctrl('C')) {
chat_send(cfd, "/b");
break;
- }
- if (ch == Ctrl('D')) {
+ } else if (ch == Ctrl('D')) {
if ((size_t)currchar < strlen(inbuf)) {
inbuf[69] = '\0';
memcpy(&inbuf[currchar], &inbuf[currchar + 1], 69 - currchar);
@@ -505,23 +478,15 @@ t_chat()
clrtoeol();
outs(&inbuf[currchar]);
}
- continue;
- }
- if (ch == Ctrl('K')) {
+ } else if (ch == Ctrl('K')) {
inbuf[currchar] = 0;
move(b_lines - 1, currchar + chatid_len);
clrtoeol();
- continue;
- }
- if (ch == Ctrl('A')) {
+ } else if (ch == Ctrl('A')) {
currchar = 0;
- continue;
- }
- if (ch == Ctrl('E')) {
+ } else if (ch == Ctrl('E')) {
currchar = strlen(inbuf);
- continue;
- }
- if (ch == Ctrl('I')) {
+ } else if (ch == Ctrl('I')) {
screenline_t *screen0 = calloc(t_lines, sizeof(screenline_t));
memcpy(screen0, big_picture, t_lines * sizeof(screenline_t));
@@ -531,9 +496,7 @@ t_chat()
free(screen0);
redoscr();
add_io(cfd, 0);
- continue;
- }
- if (ch == Ctrl('Q')) {
+ } else if (ch == Ctrl('Q')) {
print_chatid(chatid);
move(b_lines - 1, chatid_len);
outs(inbuf);
@@ -555,6 +518,7 @@ t_chat()
if (*ans == 'm') {
fileheader_t mymail;
char title[128];
+ char genbuf[200];
sethomepath(genbuf, cuser.userid);
stampfile(genbuf, &mymail);
@@ -569,7 +533,3 @@ t_chat()
}
return 0;
}
-/* -------------------------------------------------- */
-#if 0
-
-#endif
diff --git a/mbbsd/mail.c b/mbbsd/mail.c
index da078f1e..e0dced0f 100644
--- a/mbbsd/mail.c
+++ b/mbbsd/mail.c
@@ -776,7 +776,7 @@ read_new_mail(fileheader_t * fptr)
getdata(1, 0, msg_sure_ny, genbuf, 2, LCECHO);
if (genbuf[0] == 'y') {
unlink(fname);
- delmsgs[delcnt++] = idc;
+ delmsgs[delcnt++] = idc; // FIXME 一次刪太多信 out of array boundary
}
}
clear();
diff --git a/mbbsd/more.c b/mbbsd/more.c
index b4fe1948..2a476632 100644
--- a/mbbsd/more.c
+++ b/mbbsd/more.c
@@ -1,4 +1,4 @@
-/* $Id: more.c,v 1.24 2003/01/23 18:39:23 in2 Exp $ */
+/* $Id$ */
#include "bbs.h"
#define MORE_BUFSIZE 4096
#define MORE_WINSIZE 4096
@@ -117,19 +117,6 @@ more_readln(int fd, unsigned char *buf)
return bytes;
}
-/*
- * not used static int readln(FILE *fp, char *buf) { register int ch, i, len,
- * bytes, in_ansi;
- *
- * len = bytes = in_ansi = i = 0; while(len < 80 && i < ANSILINELEN && (ch =
- * getc(fp)) != EOF) { bytes++; if(ch == '\n') break; else if(ch == '\t') do
- * { buf[i++] = ' '; } while((++len & 7) && len < 80); else if(ch == '\a')
- * beep = 1; else if(ch == '\033') { if(showansi) buf[i++] = ch; in_ansi = 1;
- * } else if(in_ansi) { if(showansi) buf[i++] = ch;
- * if(!strchr("[0123456789;,", ch)) in_ansi = 0; } else if(isprint2(ch)) {
- * len++; buf[i++] = ch; } } buf[i] = '\0'; return bytes; }
- */
-
int
more(char *fpath, int promptend)
{
@@ -137,8 +124,6 @@ more(char *fpath, int promptend)
char *ptr, *word = NULL, buf[ANSILINELEN + 1];
struct stat st;
- /* rocker */
- //FILE * fp;
int fd, fsize;
unsigned int pagebreak[MAX_PAGES], pageno, lino = 0;
diff --git a/mbbsd/talk.c b/mbbsd/talk.c
index 32718676..2ef59c86 100644
--- a/mbbsd/talk.c
+++ b/mbbsd/talk.c
@@ -37,7 +37,6 @@ static char *fcolor[11] = {
};
static char save_page_requestor[40];
static char page_requestor[40];
-static char description[30];
static FILE *flog;
int
@@ -1414,14 +1413,13 @@ t_showhelp()
/* Kaede show friend description */
static char *
-friend_descript(userinfo_t * uentp)
+friend_descript(userinfo_t * uentp, char *desc_buf, int desc_buflen)
{
char *space_buf = "";
- static char desc_buf[80];
char fpath[80], name[IDLEN + 2], *desc, *ptr;
int len, flag;
FILE *fp;
- char genbuf[200];
+ char genbuf[STRLEN];
if((set_friend_bit(currutmp,uentp)|IFH)==0)
return space_buf;
@@ -1443,7 +1441,7 @@ friend_descript(userinfo_t * uentp)
}
fclose(fp);
if (flag)
- strlcpy(desc_buf, desc, sizeof(desc_buf));
+ strlcpy(desc_buf, desc, desc_buflen);
else
return space_buf;
@@ -1456,9 +1454,10 @@ friend_descript(userinfo_t * uentp)
static char *
descript(int show_mode, userinfo_t * uentp, time_t diff)
{
+ static char description[30];
switch (show_mode) {
case 1:
- return friend_descript(uentp);
+ return friend_descript(uentp, description, sizeof(description));
case 0:
return (((uentp->pager != 2 && uentp->pager != 3 && diff) ||
HAS_PERM(PERM_SYSOP)) ?
@@ -1473,13 +1472,11 @@ descript(int show_mode, userinfo_t * uentp, time_t diff)
snprintf(description, sizeof(description),
"%3d/%3d/%3d", uentp->five_win,
uentp->five_lose, uentp->five_tie);
- description[20] = 0;
return description;
case 3:
snprintf(description, sizeof(description),
"%3d/%3d/%3d", uentp->chc_win,
uentp->chc_lose, uentp->chc_tie);
- description[20] = 0;
return description;
default:
syslog(LOG_WARNING, "damn!!! what's wrong?? show_mode = %d",
@@ -1841,7 +1838,7 @@ userlist(void)
static char show_uid = 0;
static char show_board = 0;
static char show_pid = 0;
- char genbuf[256], skippickup = 0, redraw, redrawall;
+ char skippickup = 0, redraw, redrawall;
int page, offset, pickup_way, ch, leave, fri_stat;
int nfriend, myfriend, friendme, bfriend, badfriend, i;
time_t lastupdate;
@@ -2124,9 +2121,10 @@ userlist(void)
case 'b': /* broadcast */
if (cuser.uflag & FRIEND_FLAG || HAS_PERM(PERM_SYSOP)) {
+ char genbuf[60];
char ans[4];
- if (!getdata(0, 0, "廣播訊息:", genbuf, 60, DOECHO))
+ if (!getdata(0, 0, "廣播訊息:", genbuf, sizeof(genbuf), DOECHO))
break;
if (getdata(0, 0, "確定廣播? [Y]",
ans, sizeof(ans), LCECHO) &&
@@ -2268,6 +2266,7 @@ userlist(void)
case 'g':
if (HAS_PERM(PERM_LOGINOK) &&
strcmp(uentp->userid, cuser.userid) != 0) {
+ char genbuf[128];
move(b_lines - 2, 0);
prints("要給 %s 多少錢呢? ", uentp->userid);
if (getdata(b_lines - 1, 0, "[銀行轉帳]: ",