alias(recp);
cvt = convert_internet_address(user, node, recp);
- sprintf(recp, "%s@%s", user, node);
+ snprintf(recp, sizeof recp, "%s@%s", user, node);
lprintf(9, "cvt=%d, citaddr=<%s@%s>\n", cvt, user, node);
switch(cvt) {
* (This is kind of ugly. IGnet should be done using clean server-to-server
* code instead of the old style spool.)
*/
-void smtp_deliver_ignet(struct CtdlMessage *msg, char *user, char *room) {
+void smtp_deliver_ignet(struct CtdlMessage *msg, char *user, char *dest) {
struct ser_ret smr;
- char *hold_R, *hold_D;
+ char *hold_R, *hold_D, *hold_O;
FILE *fp;
+ char filename[256];
+ static int seq = 0;
+
+ lprintf(9, "smtp_deliver_ignet(msg, %s, %s)\n", user, dest);
hold_R = msg->cm_fields['R'];
hold_D = msg->cm_fields['D'];
+ hold_O = msg->cm_fields['O'];
msg->cm_fields['R'] = user;
- msg->cm_fields['D'] = room;
+ msg->cm_fields['D'] = dest;
+ msg->cm_fields['O'] = MAILROOM;
serialize_message(&smr, msg);
msg->cm_fields['R'] = hold_R;
msg->cm_fields['D'] = hold_D;
+ msg->cm_fields['O'] = hold_O;
if (smr.len != 0) {
- fp = fopen(tmpnam("./network/spoolin/"), "wb");
+ snprintf(filename, sizeof filename,
+ "./network/spoolin/%s.%04x.%04x",
+ dest, getpid(), ++seq);
+ lprintf(9, "spool file name is <%s>\n", filename);
+ fp = fopen(filename, "wb");
if (fp != NULL) {
fwrite(smr.ser, smr.len, 1, fp);
fclose(fp);
if (msg->cm_fields['A']==NULL) msg->cm_fields['A'] = strdoop(user);
if (msg->cm_fields['N']==NULL) msg->cm_fields['N'] = strdoop(node);
if (msg->cm_fields['H']==NULL) msg->cm_fields['H'] = strdoop(name);
+ if (msg->cm_fields['O']==NULL) msg->cm_fields['O'] = strdoop(MAILROOM);
/* Save the message in the queue */
msgid = CtdlSaveMsg(msg,
++successful_saves;
instr = mallok(1024);
- sprintf(instr, "Content-type: %s\n\nmsgid|%ld\nsubmitted|%ld\n"
+ snprintf(instr, 1024,
+ "Content-type: %s\n\nmsgid|%ld\nsubmitted|%ld\n"
"bounceto|%s\n",
SPOOLMIME, msgid, time(NULL),
SMTP->from );
/* Delivery over the local Citadel network (IGnet) */
if (!strcasecmp(dtype, "ignet")) {
- smtp_deliver_ignet(msg, user, room);
+ extract(user, buf, 1);
+ extract(node, buf, 2);
+ smtp_deliver_ignet(msg, user, node);
}
/* Remote delivery */
if (!strcasecmp(dtype, "remote")) {
extract(user, buf, 1);
instr = reallok(instr, strlen(instr) + 1024);
- sprintf(&instr[strlen(instr)],
+ snprintf(&instr[strlen(instr)],
+ strlen(instr) + 1024,
"remote|%s|0\n",
user);
++remote_spools;
generate_rfc822_datestamp(nowstamp, time(NULL));
body = mallok(4096);
- if (body != NULL) sprintf(body,
+ if (body != NULL) snprintf(body, 4096,
"Received: from %s\n"
" by %s;\n"
" %s\n",
lprintf(9, "Number of MX hosts for <%s> is %d\n", node, num_mxhosts);
if (num_mxhosts < 1) {
*status = 5;
- sprintf(dsn, "No MX hosts found for <%s>", node);
+ snprintf(dsn, 256, "No MX hosts found for <%s>", node);
return;
}
extract(buf, mxhosts, mx);
lprintf(9, "Trying <%s>\n", buf);
sock = sock_connect(buf, "25", "tcp");
- sprintf(dsn, "Could not connect: %s", strerror(errno));
+ snprintf(dsn, 256, "Could not connect: %s", strerror(errno));
if (sock >= 0) lprintf(9, "Connected!\n");
- if (sock < 0) sprintf(dsn, "%s", strerror(errno));
+ if (sock < 0) snprintf(dsn, 256, "%s", strerror(errno));
if (sock >= 0) break;
}
if (buf[0] != '2') {
if (buf[0] == '4') {
*status = 4;
- strcpy(dsn, &buf[4]);
+ safestrncpy(dsn, &buf[4], 1023);
goto bail;
}
else {
*status = 5;
- strcpy(dsn, &buf[4]);
+ safestrncpy(dsn, &buf[4], 1023);
goto bail;
}
}
/* At this point we know we are talking to a real SMTP server */
/* Do a HELO command */
- sprintf(buf, "HELO %s", config.c_fqdn);
+ snprintf(buf, sizeof buf, "HELO %s", config.c_fqdn);
lprintf(9, ">%s\n", buf);
sock_puts(sock, buf);
if (sock_gets(sock, buf) < 0) {
if (buf[0] != '2') {
if (buf[0] == '4') {
*status = 4;
- strcpy(dsn, &buf[4]);
+ safestrncpy(dsn, &buf[4], 1023);
goto bail;
}
else {
*status = 5;
- strcpy(dsn, &buf[4]);
+ safestrncpy(dsn, &buf[4], 1023);
goto bail;
}
}
/* HELO succeeded, now try the MAIL From: command */
- sprintf(buf, "MAIL From: %s", mailfrom);
+ snprintf(buf, sizeof buf, "MAIL From: %s", mailfrom);
lprintf(9, ">%s\n", buf);
sock_puts(sock, buf);
if (sock_gets(sock, buf) < 0) {
if (buf[0] != '2') {
if (buf[0] == '4') {
*status = 4;
- strcpy(dsn, &buf[4]);
+ safestrncpy(dsn, &buf[4], 1023);
goto bail;
}
else {
*status = 5;
- strcpy(dsn, &buf[4]);
+ safestrncpy(dsn, &buf[4], 1023);
goto bail;
}
}
/* MAIL succeeded, now try the RCPT To: command */
- sprintf(buf, "RCPT To: %s", addr);
+ snprintf(buf, sizeof buf, "RCPT To: %s", addr);
lprintf(9, ">%s\n", buf);
sock_puts(sock, buf);
if (sock_gets(sock, buf) < 0) {
if (buf[0] != '2') {
if (buf[0] == '4') {
*status = 4;
- strcpy(dsn, &buf[4]);
+ safestrncpy(dsn, &buf[4], 1023);
goto bail;
}
else {
*status = 5;
- strcpy(dsn, &buf[4]);
+ safestrncpy(dsn, &buf[4], 1023);
goto bail;
}
}
if (buf[0] != '3') {
if (buf[0] == '4') {
*status = 3;
- strcpy(dsn, &buf[4]);
+ safestrncpy(dsn, &buf[4], 1023);
goto bail;
}
else {
*status = 5;
- strcpy(dsn, &buf[4]);
+ safestrncpy(dsn, &buf[4], 1023);
goto bail;
}
}
if (buf[0] != '2') {
if (buf[0] == '4') {
*status = 4;
- strcpy(dsn, &buf[4]);
+ safestrncpy(dsn, &buf[4], 1023);
goto bail;
}
else {
*status = 5;
- strcpy(dsn, &buf[4]);
+ safestrncpy(dsn, &buf[4], 1023);
goto bail;
}
}
/* We did it! */
- strcpy(dsn, &buf[4]);
+ safestrncpy(dsn, &buf[4], 1023);
*status = 2;
lprintf(9, ">QUIT\n");
bmsg->cm_fields['N'] = strdoop(config.c_nodename);
if (give_up) bmsg->cm_fields['M'] = strdoop(
-"A message you sent could not be delivered to some or all of its recipients.\n"
-"The following addresses were undeliverable:\n\n"
-);
-
- else bmsg->cm_fields['M'] = strdoop(
"A message you sent could not be delivered to some or all of its recipients\n"
"due to prolonged unavailability of its destination(s).\n"
"Giving up on the following addresses:\n\n"
+);
+
+ else bmsg->cm_fields['M'] = strdoop(
+"A message you sent could not be delivered to some or all of its recipients.\n"
+"The following addresses were undeliverable:\n\n"
);
/*
msg->cm_anon_type = MES_NORMAL;
msg->cm_format_type = FMT_RFC822;
msg->cm_fields['M'] = malloc(strlen(instr)+256);
- sprintf(msg->cm_fields['M'],
+ snprintf(msg->cm_fields['M'],
+ strlen(instr)+256,
"Content-type: %s\n\n%s\nattempted|%ld\n",
SPOOLMIME, instr, time(NULL) );
phree(instr);
{
SYM_SMTP = CtdlGetDynamicSymbol();
SYM_SMTP_RECP = CtdlGetDynamicSymbol();
- CtdlRegisterServiceHook(SMTP_PORT,
+
+ CtdlRegisterServiceHook(config.c_smtp_port, /* On the net... */
+ NULL,
smtp_greeting,
smtp_command_loop);
+
+ CtdlRegisterServiceHook(0, /* ...and locally */
+ "smtp.socket",
+ smtp_greeting,
+ smtp_command_loop);
+
create_room(SMTP_SPOOLOUT_ROOM, 3, "", 0);
CtdlRegisterSessionHook(smtp_do_queue, EVT_TIMER);
return "$Id$";