]> arthur.barton.de Git - netatalk.git/blobdiff - etc/atalkd/rtmp.c
Big configure.in cleanup
[netatalk.git] / etc / atalkd / rtmp.c
index e35b89a7bb121d2b468c4c74cb841652e74b1b34..727562b0bd31b5dcfcbd7131c8283c3c48d93b6f 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * $Id: rtmp.c,v 1.7 2001-08-03 22:12:54 srittau Exp $
+ * $Id: rtmp.c,v 1.17 2009-12-08 03:21:16 didg Exp $
  *
  * Copyright (c) 1990,1993 Regents of The University of Michigan.
  * All Rights Reserved. See COPYRIGHT.
 
 #include <stdlib.h>
 #include <string.h>
-#include <sys/syslog.h>
+#include <errno.h>
+#include <atalk/logger.h>
 #include <sys/types.h>
 #include <sys/param.h>
 #include <sys/socket.h>
 #include <sys/ioctl.h>
+#ifdef TRU64
+#include <sys/mbuf.h>
+#include <net/route.h>
+#endif /* TRU64 */
 #include <net/if.h>
 #include <net/route.h>
 #include <netatalk/endian.h>
@@ -38,8 +43,7 @@
 #include "route.h"
 #include "main.h"
 
-void rtmp_delzonemap(rtmp)
-    struct rtmptab *rtmp;
+void rtmp_delzonemap(struct rtmptab *rtmp)
 {
     struct list                *lz, *flz, *lr, *flr;
     struct ziptab      *zt;
@@ -92,9 +96,7 @@ void rtmp_delzonemap(rtmp)
 /*
  * Complete configuration for phase 1 interface using RTMP information.
  */
-static int rtmp_config( rh, iface )
-    struct rtmp_head   *rh;
-    struct interface   *iface;
+static int rtmp_config( struct rtmp_head *rh, struct interface *iface)
 {
     extern int         stabletimer;
     int cc;
@@ -104,7 +106,7 @@ static int rtmp_config( rh, iface )
      * configuration with RTMP.
      */
     if ( iface->i_flags & IFACE_PHASE2 ) {
-       syslog( LOG_INFO, "rtmp_config ignoring data" );
+       LOG(log_info, logtype_atalkd, "rtmp_config ignoring data" );
        return 0;
     }
 
@@ -114,7 +116,7 @@ static int rtmp_config( rh, iface )
     if ( rh->rh_net != iface->i_addr.sat_addr.s_net ) {
        if (( iface->i_flags & IFACE_SEED ) &&
                rh->rh_net != iface->i_caddr.sat_addr.s_net) {
-           syslog( LOG_ERR, "rtmp_config net mismatch %u != %u",
+           LOG(log_error, logtype_atalkd, "rtmp_config net mismatch %u != %u",
                    ntohs( rh->rh_net ),
                    ntohs( iface->i_addr.sat_addr.s_net ));
            return 1;
@@ -137,12 +139,13 @@ static int rtmp_config( rh, iface )
       return -1;
 
     if (cc) {
-       syslog( LOG_ERR, "rtmp_config: can't route %u.%u to loopback: %m",
+       LOG(log_error, logtype_atalkd, "rtmp_config: can't route %u.%u to loopback: %s",
                ntohs( iface->i_addr.sat_addr.s_net ),
-               iface->i_addr.sat_addr.s_node );
+               iface->i_addr.sat_addr.s_node,
+               strerror(errno) );
     }
 
-    syslog( LOG_INFO, "rtmp_config configured %s", iface->i_name );
+    LOG(log_info, logtype_atalkd, "rtmp_config configured %s", iface->i_name );
     iface->i_flags |= IFACE_CONFIG;
     if ( iface == ciface ) {
        ciface = ciface->i_next;
@@ -156,8 +159,7 @@ static int rtmp_config( rh, iface )
  * Delete rtmp from the per-interface in-use table, remove all
  * zone references, and remove the route from the kernel.
  */
-static void rtmp_delinuse( rtmp )
-    struct rtmptab     *rtmp;
+static void rtmp_delinuse(struct rtmptab *rtmp)
 {
     struct rtmptab     *irt;
 
@@ -191,8 +193,7 @@ static void rtmp_delinuse( rtmp )
 /*
  * Add rtmp to the per-interface in-use table.  No verification is done...
  */
-static void rtmp_addinuse( rtmp )
-    struct rtmptab     *rtmp;
+static void rtmp_addinuse( struct rtmptab *rtmp)
 {
     struct rtmptab     *irt;
 
@@ -219,8 +220,7 @@ static void rtmp_addinuse( rtmp )
  * at this point?  What do we do if we get several copies of a route in
  * an RTMP packet?
  */
-static int rtmp_copyzones( to, from )
-    struct rtmptab     *to, *from;
+static int rtmp_copyzones( struct rtmptab *to,struct rtmptab *from)
 {
     struct list                *lz, *lr;
 
@@ -237,7 +237,7 @@ static int rtmp_copyzones( to, from )
            }
        }
        if ( lr == NULL ) {
-           syslog( LOG_ERR, "rtmp_copyzones z -> r without r -> z, abort" );
+           LOG(log_error, logtype_atalkd, "rtmp_copyzones z -> r without r -> z, abort" );
            return -1;
        }
     }
@@ -250,12 +250,11 @@ static int rtmp_copyzones( to, from )
  * Remove rtmp from the in-use table and the per-gate table.
  * Free any associated space.
  */
-void rtmp_free( rtmp )
-    struct rtmptab     *rtmp;
+void rtmp_free( struct rtmptab *rtmp)
 {
     struct gate                *gate;
 
-    syslog(LOG_INFO, "rtmp_free: %u-%u", ntohs(rtmp->rt_firstnet),
+    LOG(log_info, logtype_atalkd, "rtmp_free: %u-%u", ntohs(rtmp->rt_firstnet),
           ntohs(rtmp->rt_lastnet));
     if ( rtmp->rt_iprev ) {
        rtmp_delinuse( rtmp );
@@ -288,14 +287,13 @@ void rtmp_free( rtmp )
  * Find a replacement for "replace".  If we can't find a replacement,
  * return 1.  If we do find a replacement, return 0. -1 on error.
  */
-int rtmp_replace( replace )
-    struct rtmptab     *replace;
+int rtmp_replace(struct rtmptab *replace)
 {
     struct interface   *iface;
     struct gate                *gate;
     struct rtmptab     *rtmp, *found = NULL;
 
-    syslog(LOG_INFO, "rtmp_replace %u-%u", ntohs(replace->rt_firstnet),
+    LOG(log_info, logtype_atalkd, "rtmp_replace %u-%u", ntohs(replace->rt_firstnet),
           ntohs(replace->rt_lastnet));
     for ( iface = interfaces; iface; iface = iface->i_next ) {
         if ((replace->rt_iface != iface) && 
@@ -333,8 +331,7 @@ int rtmp_replace( replace )
 }
 
 
-static int rtmp_new( rtmp )
-    struct rtmptab     *rtmp;
+static int rtmp_new(struct rtmptab *rtmp)
 {
     struct interface   *i;
     struct rtmptab     *r;
@@ -370,7 +367,7 @@ static int rtmp_new( rtmp )
     if ( i ) { /* can we get here without r being set? */
        if ( r->rt_firstnet != rtmp->rt_firstnet ||
                r->rt_lastnet != rtmp->rt_lastnet ) {
-           syslog( LOG_INFO, "rtmp_new netrange mismatch %u-%u != %u-%u",
+           LOG(log_info, logtype_atalkd, "rtmp_new netrange mismatch %u-%u != %u-%u",
                    ntohs( r->rt_firstnet ), ntohs( r->rt_lastnet ),
                    ntohs( rtmp->rt_firstnet ), ntohs( rtmp->rt_lastnet ));
            return 1;
@@ -397,16 +394,13 @@ static int rtmp_new( rtmp )
 }
 
 
-int rtmp_packet( ap, from, data, len )
-    struct atport      *ap;
-    struct sockaddr_at *from;
-    char               *data;
-    int                        len;
+int rtmp_packet(struct atport *ap, struct sockaddr_at *from, char *data, int len)
 {
     struct rtmp_head   rh;
     struct rtmp_tuple  rt, xrt;
     struct gate                *gate;
     struct interface   *iface;
+    struct interface   *iface2;
     struct rtmptab     *rtmp;
     char               *end, packet[ ATP_BUFSIZ ];
     int                 cc;
@@ -414,12 +408,27 @@ int rtmp_packet( ap, from, data, len )
     end = data + len;
 
     if ( data >= end ) {
-       syslog( LOG_INFO, "rtmp_packet no data" );
+       LOG(log_info, logtype_atalkd, "rtmp_packet no data" );
        return 1;
     }
 
     iface = ap->ap_iface;
 
+    /* linux 2.6 sends broadcast queries to the first available socket 
+       (in our case the last configured) 
+       try to find the right one.
+       Note: now a misconfigured or plugged router can broadcast
+       a wrong route
+    */
+    for ( iface2 = interfaces; iface2; iface2 = iface2->i_next ) {
+        if ( iface2->i_rt && from->sat_addr.s_net >= iface2->i_rt->rt_firstnet && 
+                from->sat_addr.s_net <= iface2->i_rt->rt_lastnet) 
+        {
+              iface = iface2;
+        }
+    }
+    /* end of linux 2.6 workaround */
+    
     /* ignore our own packets */
     if ( from->sat_addr.s_net == iface->i_addr.sat_addr.s_net &&
            from->sat_addr.s_node == iface->i_addr.sat_addr.s_node  ) {
@@ -432,7 +441,7 @@ int rtmp_packet( ap, from, data, len )
         * Response and Data.
         */
        if ( data + sizeof( struct rtmprdhdr ) > end ) {
-           syslog( LOG_INFO, "rtmp_packet no data header" );
+           LOG(log_info, logtype_atalkd, "rtmp_packet no data header" );
            return 1;
        }
        memcpy( &rh, data, sizeof( struct rtmprdhdr ));
@@ -440,13 +449,13 @@ int rtmp_packet( ap, from, data, len )
 
        /* check rh address against from address */
        if ( rh.rh_nodelen != 8 ) {
-           syslog( LOG_INFO, "rtmp_packet bad node len (%d)", rh.rh_nodelen );
+           LOG(log_info, logtype_atalkd, "rtmp_packet bad node len (%d)", rh.rh_nodelen );
            return 1;
        }
        if (( from->sat_addr.s_net != 0 && 
              from->sat_addr.s_net != rh.rh_net ) ||
              from->sat_addr.s_node != rh.rh_node ) {
-           syslog( LOG_INFO, "rtmp_packet address mismatch" );
+           LOG(log_info, logtype_atalkd, "rtmp_packet address mismatch" );
            return 1;
        }
 
@@ -454,24 +463,25 @@ int rtmp_packet( ap, from, data, len )
            if ( iface->i_flags & IFACE_NOROUTER ) {
                /* remove addr to loopback route */
                if ((cc = looproute( iface, RTMP_DEL )) < 0) {
-                 syslog(LOG_ERR, "rtmp_packet: looproute");
+                 LOG(log_error, logtype_atalkd, "rtmp_packet: looproute");
                  return -1;
                } 
 
                if (cc)
-                 syslog( LOG_ERR, "rtmp_packet: can't remove loopback: %m" );
+                 LOG(log_error, logtype_atalkd, "rtmp_packet: can't remove loopback: %s",
+                         strerror(errno) );
 
                iface->i_flags &= ~IFACE_NOROUTER;
                iface->i_time = 0;
-               syslog( LOG_INFO, "rtmp_packet router has become available" );
+               LOG(log_info, logtype_atalkd, "rtmp_packet router has become available" );
            }
            if ( iface->i_flags & IFACE_PHASE1 ) {
              if (rtmp_config( &rh, iface ) < 0) {
-                 syslog(LOG_ERR, "rtmp_packet: rtmp_config");
+                 LOG(log_error, logtype_atalkd, "rtmp_packet: rtmp_config");
                  return -1;
              }
            } else if (zip_getnetinfo( iface ) < 0) {
-             syslog(LOG_ERR, "rtmp_packet: zip_getnetinfo");
+             LOG(log_error, logtype_atalkd, "rtmp_packet: zip_getnetinfo");
              return -1;
            }
            return 0;
@@ -485,7 +495,7 @@ int rtmp_packet( ap, from, data, len )
         * Parse first tuple.  For phase 2, verify that net is correct.
         */
        if ( data + SZ_RTMPTUPLE > end ) {
-           syslog( LOG_INFO, "rtmp_packet missing first tuple" );
+           LOG(log_info, logtype_atalkd, "rtmp_packet missing first tuple" );
            return 1;
        }
        memcpy( &rt, data, SZ_RTMPTUPLE );
@@ -493,7 +503,7 @@ int rtmp_packet( ap, from, data, len )
 
        if ( rt.rt_net == 0 ) {
            if ( rt.rt_dist != 0x82 ) {
-               syslog( LOG_INFO, "rtmp_packet bad phase 1 version" );
+               LOG(log_info, logtype_atalkd, "rtmp_packet bad phase 1 version" );
                return 1;
            }
 
@@ -503,21 +513,21 @@ int rtmp_packet( ap, from, data, len )
             * no extended tuples in this packet.
             */
            if ( data + SZ_RTMPTUPLE > end ) {
-               syslog( LOG_INFO, "rtmp_packet missing second tuple" );
+               LOG(log_info, logtype_atalkd, "rtmp_packet missing second tuple" );
                return 1;
            }
            memcpy( &rt, data, SZ_RTMPTUPLE );
            data += SZ_RTMPTUPLE;
        } else if ( rt.rt_dist & 0x80 ) {
            if ( data + SZ_RTMPTUPLE > end ) {
-               syslog( LOG_INFO, "rtmp_packet missing first range-end" );
+               LOG(log_info, logtype_atalkd, "rtmp_packet missing first range-end" );
                return 1;
            }
            memcpy( &xrt, data, SZ_RTMPTUPLE );
            data += SZ_RTMPTUPLE;
 
            if ( xrt.rt_dist != 0x82 ) {
-               syslog( LOG_INFO, "rtmp_packet bad phase 2 version" );
+               LOG(log_info, logtype_atalkd, "rtmp_packet bad phase 2 version" );
                return 1;
            }
 
@@ -526,7 +536,7 @@ int rtmp_packet( ap, from, data, len )
             */
            if ( rt.rt_net != iface->i_rt->rt_firstnet ||
                    xrt.rt_net != iface->i_rt->rt_lastnet ) {
-               syslog( LOG_INFO, "rtmp_packet interface mismatch" );
+               LOG(log_info, logtype_atalkd, "rtmp_packet interface mismatch" );
                return 1;
            }
        } else {
@@ -537,12 +547,12 @@ int rtmp_packet( ap, from, data, len )
             * might just as well check it.
            if ( rt.rt_net != iface->i_rt->rt_firstnet ||
                    rt.rt_net != iface->i_rt->rt_lastnet ) {
-               syslog( LOG_INFO, "rtmp_packet phase 1 interface mismatch" );
+               LOG(log_info, logtype_atalkd, "rtmp_packet phase 1 interface mismatch" );
                return 1;
            }
             */
 #else /* PHASE1NET */
-           syslog( LOG_INFO, "rtmp_packet bad first tuple" );
+           LOG(log_info, logtype_atalkd, "rtmp_packet bad first tuple" );
            return 1;
 #endif /* PHASE1NET */
        }
@@ -557,20 +567,20 @@ int rtmp_packet( ap, from, data, len )
            }
        }
        if ( !gate ) {  /* new gateway */
-           if (( gate = (struct gate *)malloc( sizeof( struct gate ))) == 0 ) {
-               syslog( LOG_ERR, "rtmp_packet: malloc: %m" );
+           if (( gate = (struct gate *)malloc( sizeof( struct gate ))) == NULL ) {
+               LOG(log_error, logtype_atalkd, "rtmp_packet: malloc: %s", strerror(errno) );
                return -1;
            }
            gate->g_next = iface->i_gate;
-           gate->g_prev = 0;
-           gate->g_rt = 0;
+           gate->g_prev = NULL;
+           gate->g_rt = NULL;
            gate->g_iface = iface;      /* need this? */
            gate->g_sat = *from;
            if ( iface->i_gate ) {
                iface->i_gate->g_prev = gate;
            }
            iface->i_gate = gate;
-           syslog( LOG_INFO, "rtmp_packet gateway %u.%u up",
+           LOG(log_info, logtype_atalkd, "rtmp_packet gateway %u.%u up",
                    ntohs( gate->g_sat.sat_addr.s_net ),
                    gate->g_sat.sat_addr.s_node );
        }
@@ -606,29 +616,29 @@ int rtmp_packet( ap, from, data, len )
                 * ugly.)
                 */
                if ( rtmp->rt_firstnet != rt.rt_net ) {
-                   syslog( LOG_INFO, "rtmp_packet firstnet mismatch %u!=%u",
+                   LOG(log_info, logtype_atalkd, "rtmp_packet firstnet mismatch %u!=%u",
                            ntohs( rtmp->rt_firstnet ), ntohs( rt.rt_net ));
                    return 1;
                }
                if ( rt.rt_dist & 0x80 ) {
                    if (( rtmp->rt_flags & RTMPTAB_EXTENDED ) == 0 ) {
-                       syslog( LOG_INFO, "rtmp_packet extended mismatch %u",
+                       LOG(log_info, logtype_atalkd, "rtmp_packet extended mismatch %u",
                                ntohs( rtmp->rt_firstnet ));
                        return 1;
                    }
                    if ( rtmp->rt_lastnet != xrt.rt_net ) {
-                       syslog( LOG_INFO, "rtmp_packet lastnet mismatch %u!=%u",
+                       LOG(log_info, logtype_atalkd, "rtmp_packet lastnet mismatch %u!=%u",
                            ntohs( rtmp->rt_lastnet ), ntohs( xrt.rt_net ));
                        return 1;
                    }
                } else {
                    if ( rtmp->rt_flags & RTMPTAB_EXTENDED ) {
-                       syslog( LOG_INFO, "rtmp_packet !extended mismatch %u",
+                       LOG(log_info, logtype_atalkd, "rtmp_packet !extended mismatch %u",
                                ntohs( rtmp->rt_firstnet ));
                        return 1;
                    }
                    if ( rtmp->rt_lastnet != rt.rt_net ) {
-                       syslog( LOG_INFO, "rtmp_packet lastnet mismatch %u!=%u",
+                       LOG(log_info, logtype_atalkd, "rtmp_packet lastnet mismatch %u!=%u",
                            ntohs( rtmp->rt_lastnet ), ntohs( rt.rt_net ));
                        return 1;
                    }
@@ -666,7 +676,7 @@ int rtmp_packet( ap, from, data, len )
                                rtmp->rt_hops = ( rt.rt_dist & 0x7f ) + 1;
                            }
                            if (rtmp_replace( rtmp ) < 0) {
-                             syslog(LOG_ERR, "rtmp_packet: rtmp_replace");
+                             LOG(log_error, logtype_atalkd, "rtmp_packet: rtmp_replace");
                              return -1;
                            }
                        }
@@ -674,7 +684,7 @@ int rtmp_packet( ap, from, data, len )
                        rtmp->rt_hops = ( rt.rt_dist & 0x7f ) + 1;
                        if ( rtmp->rt_hops > ( rt.rt_dist & 0x7f ) + 1 ) {
                          if (rtmp_new( rtmp ) < 0) {
-                             syslog(LOG_ERR, "rtmp_packet: rtmp_new");
+                             LOG(log_error, logtype_atalkd, "rtmp_packet: rtmp_new");
                              return -1;
                          }
                        }
@@ -686,18 +696,18 @@ int rtmp_packet( ap, from, data, len )
                 * we're not likely to be asked for the same tuple twice
                 * in a row.
                 */
-               if ( rtmp->rt_next != 0 ) {
+               if ( rtmp->rt_next != NULL ) {
                    gate->g_rt->rt_prev->rt_next = gate->g_rt;
                    gate->g_rt = rtmp->rt_next;
-                   rtmp->rt_next = 0;
+                   rtmp->rt_next = NULL;
                }
            } else if (( rt.rt_dist & 0x7f ) + 1 > RTMPHOPS_MAX ) {
-               syslog( LOG_INFO, "rtmp_packet bad hop count from %u.%u for %u",
+               LOG(log_info, logtype_atalkd, "rtmp_packet bad hop count from %u.%u for %u",
                        ntohs( from->sat_addr.s_net ), from->sat_addr.s_node,
                        ntohs( rt.rt_net ));
            } else {            /* new for router */
                if (( rtmp = newrt(iface)) == NULL ) {
-                   syslog( LOG_ERR, "rtmp_packet: newrt: %m" );
+                   LOG(log_error, logtype_atalkd, "rtmp_packet: newrt: %s", strerror(errno) );
                    return -1;
                }
                rtmp->rt_firstnet = rt.rt_net;
@@ -714,7 +724,7 @@ int rtmp_packet( ap, from, data, len )
                /*
                 * Add rtmptab entry to end of list (leave head alone).
                 */
-               if ( gate->g_rt == 0 ) {
+               if ( gate->g_rt == NULL ) {
                    rtmp->rt_prev = rtmp;
                    gate->g_rt = rtmp;
                } else {
@@ -724,7 +734,7 @@ int rtmp_packet( ap, from, data, len )
                }
                
                if (rtmp_new( rtmp ) < 0) {
-                   syslog(LOG_ERR, "rtmp_packet: rtmp_new");
+                   LOG(log_error, logtype_atalkd, "rtmp_packet: rtmp_new");
                    return -1;
                }
            }
@@ -736,7 +746,7 @@ int rtmp_packet( ap, from, data, len )
            data += SZ_RTMPTUPLE;
            if ( rt.rt_dist & 0x80 ) {
                if ( data + SZ_RTMPTUPLE > end ) {
-                   syslog( LOG_INFO, "rtmp_packet missing range-end" );
+                   LOG(log_info, logtype_atalkd, "rtmp_packet missing range-end" );
                    return 1;
                }
                memcpy( &xrt, data, SZ_RTMPTUPLE );
@@ -748,7 +758,7 @@ int rtmp_packet( ap, from, data, len )
         * Make sure we've processed the whole packet.
         */
        if ( data != end ) {
-           syslog( LOG_INFO, "rtmp_packet length and count mismatch" );
+           LOG(log_info, logtype_atalkd, "rtmp_packet length and count mismatch" );
        }
        break;
 
@@ -757,7 +767,7 @@ int rtmp_packet( ap, from, data, len )
         * Request and RDR.
         */
         if (((iface->i_flags & IFACE_ISROUTER) == 0) ||
-           iface->i_rt->rt_zt == 0 ||
+           iface->i_rt->rt_zt == NULL ||
            ( iface->i_flags & IFACE_CONFIG ) == 0 ) {
            return 0;
        }
@@ -784,7 +794,7 @@ int rtmp_packet( ap, from, data, len )
            if ( sendto( ap->ap_fd, packet, data - packet, 0,
                    (struct sockaddr *)from,
                    sizeof( struct sockaddr_at )) < 0 ) {
-               syslog( LOG_ERR, "as_timer sendto: %m" );
+               LOG(log_error, logtype_atalkd, "as_timer sendto: %s", strerror(errno) );
            }
        } else if ( *data == 2 || *data == 3 ) {
 #ifdef DEBUG
@@ -793,13 +803,13 @@ int rtmp_packet( ap, from, data, len )
                    from->sat_addr.s_node );
 #endif /* DEBUG */
        } else {
-           syslog( LOG_INFO, "rtmp_packet unknown request from %u.%u\n",
+           LOG(log_info, logtype_atalkd, "rtmp_packet unknown request from %u.%u",
                    ntohs( from->sat_addr.s_net ), from->sat_addr.s_node );
        }
        break;
 
     default :
-       syslog( LOG_INFO, "rtmp_packet bad ddp type from %u.%u",
+       LOG(log_info, logtype_atalkd, "rtmp_packet bad ddp type from %u.%u",
                    ntohs( from->sat_addr.s_net ), from->sat_addr.s_node );
        return 0;
     }
@@ -807,22 +817,21 @@ int rtmp_packet( ap, from, data, len )
     return 0;
 }
 
-int rtmp_request( iface )
-    struct interface   *iface;
+int rtmp_request( struct interface *iface)
 {
     struct sockaddr_at sat;
     struct atport      *ap;
     char               *data, packet[ 2 ];
 
-    syslog( LOG_INFO, "rtmp_request for %s", iface->i_name );
+    LOG(log_info, logtype_atalkd, "rtmp_request for %s", iface->i_name );
 
     for ( ap = iface->i_ports; ap; ap = ap->ap_next ) {
        if ( ap->ap_packet == rtmp_packet ) {
            break;
        }
     }
-    if ( ap == 0 ) {
-       syslog( LOG_ERR, "rtmp_request can't find rtmp socket!" );
+    if ( ap == NULL ) {
+       LOG(log_error, logtype_atalkd, "rtmp_request can't find rtmp socket!" );
        return -1;
     }
 
@@ -843,26 +852,24 @@ int rtmp_request( iface )
     sat.sat_port = ap->ap_port;
     if ( sendto( ap->ap_fd, packet, data - packet, 0, (struct sockaddr *)&sat,
            sizeof( struct sockaddr_at )) < 0 ) {
-       syslog( LOG_ERR, "rtmp_request sendto: %m" );
+       LOG(log_error, logtype_atalkd, "rtmp_request sendto: %s", strerror(errno) );
        return -1;
     }
     return 0;
 }
 
 
-int looproute( iface, cmd )
-    struct interface   *iface;
-    unsigned int       cmd;
+int looproute(struct interface *iface, unsigned int cmd)
 {
     struct sockaddr_at dst, loop;
 
     if ( cmd == RTMP_DEL && ( iface->i_flags & IFACE_LOOP ) == 0 ) {
-       syslog( LOG_ERR, "looproute panic no route" );
+       LOG(log_error, logtype_atalkd, "looproute panic no route" );
        return -1;
     }
 
     if ( cmd == RTMP_ADD && ( iface->i_flags & IFACE_LOOP )) {
-       syslog( LOG_ERR, "looproute panic two routes" );
+       LOG(log_error, logtype_atalkd, "looproute panic two routes" );
        return -1;
     }
 
@@ -905,9 +912,7 @@ int looproute( iface, cmd )
     return( 0 );
 }
 
-int gateroute( command, rtmp )
-    unsigned int       command;
-    struct rtmptab     *rtmp;
+int gateroute(unsigned int command, struct rtmptab *rtmp)
 {
     struct sockaddr_at dst, gate;
     unsigned short     net;
@@ -951,8 +956,9 @@ int gateroute( command, rtmp )
                    (struct sockaddr *) &dst,
                    (struct sockaddr *) &gate,
                    RTF_UP | RTF_GATEWAY )) {
-           syslog( LOG_ERR, "route: %u -> %u.%u: %m", net,
-                   ntohs( gate.sat_addr.s_net ), gate.sat_addr.s_node );
+           LOG(log_error, logtype_atalkd, "route: %u -> %u.%u: %s", net,
+                   ntohs( gate.sat_addr.s_net ), gate.sat_addr.s_node,
+                   strerror(errno) );
            continue;
        }
 #else /* ! BSD4_4 */
@@ -960,8 +966,8 @@ int gateroute( command, rtmp )
                    (struct sockaddr_at *) &dst,
                    (struct sockaddr_at *) &gate,
                    RTF_UP | RTF_GATEWAY )) {
-           syslog( LOG_ERR, "route: %u -> %u.%u: %m", net,
-                   ntohs( gate.sat_addr.s_net ), gate.sat_addr.s_node );
+           LOG(log_error, logtype_atalkd, "route: %u -> %u.%u: %s", net,
+                   ntohs( gate.sat_addr.s_net ), gate.sat_addr.s_node, strerror(errno) );
            continue;
        }
 #endif /* ! BSD4_4 */
@@ -982,8 +988,8 @@ newrt(const struct interface *iface)
 {
     struct rtmptab     *rtmp;
 
-    if (( rtmp = (struct rtmptab *)calloc(1, sizeof(struct rtmptab))) == 0 ) {
-       return( 0 );
+    if (( rtmp = (struct rtmptab *)calloc(1, sizeof(struct rtmptab))) == NULL ) {
+       return( NULL );
     }
 
     rtmp->rt_iface = iface;