changelog: Finalise 2.2

[innduct.git] / conn.c

diff --git a/conn.c b/conn.c
index d20b342d4d22f79df61f9916337af9140e127eff..814adc8b646cf6b00ee0b970f416bb461f0fdf16 100644 (file)
--- a/conn.c
+++ b/conn.c
@@ -3,25 +3,9 @@
  *  tailing reliable realtime streaming feeder for inn
  *  conn.c - connection establishment and teardown
  *
- *  Copyright (C) 2010 Ian Jackson <ijackson@chiark.greenend.org.uk>
- * 
- *  This program is free software: you can redistribute it and/or modify
- *  it under the terms of the GNU General Public License as published by
- *  the Free Software Foundation, either version 3 of the License, or
- *  (at your option) any later version.
- * 
- *  This program is distributed in the hope that it will be useful,
- *  but WITHOUT ANY WARRANTY; without even the implied warranty of
- *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- *  GNU General Public License for more details.
- * 
- *  You should have received a copy of the GNU General Public License
- *  along with this program.  If not, see <http://www.gnu.org/licenses/>.
- *
- *  (I believe that when you compile and link this as part of the inn2
- *  build, with the Makefile runes I have provided, all the libraries
- *  and files which end up included in innduct are licence-compatible
- *  with GPLv3.  If not then please let me know.  -Ian Jackson.)
+ *  Copyright Ian Jackson <ijackson@chiark.greenend.org.uk>
+ *  and contributors; see LICENCE.txt.
+ *  SPDX-License-Identifier: GPL-3.0-or-later
  */
 
 #include "innduct.h"
@@ -293,8 +277,8 @@ static void *connchild_event(oop_source *lp, int fd, oop_event e, void *u) {
 
 #define CHK(field, val)                                                           \
   if (h->cmsg_##field != val) {                                                   \
-    crash("connect: child sent cmsg with cmsg_" #field "=%d, expected %d", \
-         h->cmsg_##field, val);                                           \
+    crash("connect: child sent cmsg with cmsg_" #field "=%ld, expected %ld", \
+         (long)h->cmsg_##field, (long)val);                               \
     goto x;                                                               \
   }
   CHK(level, SOL_SOCKET);
@@ -327,7 +311,7 @@ static void *connchild_event(oop_source *lp, int fd, oop_event e, void *u) {
   loop->on_fd(loop, conn->fd, OOP_EXCEPTION, conn_exception, conn);
   conn->rd= oop_rd_new_fd(loop,conn->fd, 0, 0); /* sets nonblocking, too */
   if (!conn->fd) crash("oop_rd_new_fd conn failed (fd=%d)",conn->fd);
-  int r= oop_rd_read(conn->rd, &peer_rd_style, NNTP_STRLEN,
+  int r= oop_rd_read(conn->rd, &peer_rd_style, NNTP_MAXLEN_COMMAND+1,
                     &peer_rd_ok, conn,
                     &peer_rd_err, conn);
   if (r) syscrash("oop_rd_read for peer (fd=%d)",conn->fd);
@@ -370,13 +354,15 @@ void connect_start(void) {
 
   if (!connecting_child) {
     FILE *cn_from, *cn_to;
-    char buf[NNTP_STRLEN+100];
+    char buf[NNTP_MAXLEN_COMMAND+100];
     int exitstatus= CONNCHILD_ESTATUS_NOSTREAM;
 
     xclose(socks[0], "(in child) parent's connection fdpass socket",0);
 
     alarm(connection_setup_timeout);
-    if (NNTPconnect((char*)remote_host, port, &cn_from, &cn_to, buf) < 0) {
+    buf[sizeof(buf)-1] = 0;
+    if (NNTPconnect(remote_host, port, &cn_from, &cn_to,
+                   buf, sizeof(buf)-1) < 0) {
       int l= strlen(buf);
       int stripped=0;
       while (l>0) {