be more carful wrt. signed vs. unsigned argument passing
[paraslash.git] / ortp_send.c
index 4dd19d7..65c6841 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2005-2006 Andre Noll <maan@systemlinux.org>
+ * Copyright (C) 2005-2007 Andre Noll <maan@systemlinux.org>
  *
  *     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
 /** \file ortp_send.c para_server's ortp sender */
 
 #include <ortp/ortp.h>
+#include <ortp/port.h>
 
 #include "server.cmdline.h"
 #include "server.h"
-#include "afs.h"
+#include "vss.h"
 #include "send.h"
 #include "list.h"
 #include "ortp.h"
@@ -39,9 +40,9 @@ struct ortp_target {
 /** whether the ortp sender is activated */
        int status;
 /** the ortp timestamp increases by this amount */
-       int chunk_ts;
+       uint32_t chunk_ts;
 /** the currently used timestamp for this target */
-       int last_ts;
+       uint32_t last_ts;
 /** the position of this target in the list of targets */
        struct list_head node;
 /** the UDP port */
@@ -67,19 +68,21 @@ static void ortp_delete_target(struct ortp_target *ot, const char *msg)
        free(ot);
 }
 
-static void ortp_send_buf(char *buf, int len, long unsigned chunks_sent)
+static void ortp_send_buf(char *buf, size_t len, long unsigned chunks_sent)
 {
        struct ortp_target *ot, *tmp;
-       int ret;
+       int ret, ortp_len = len; /* rtp_session_send_with_ts expects int */
 
+       if (ortp_len < 0)
+               return;
        list_for_each_entry_safe(ot, tmp, &targets, node) {
-               int ts;
+               uint32_t ts;
                if (!ot->session)
                        continue;
                WRITE_CHUNK_TS(buf, ot->chunk_ts);
                ts = ot->chunk_ts * chunks_sent;
                ret = rtp_session_send_with_ts(ot->session,
-                       (unsigned char*)buf, len, ts);
+                       (unsigned char*)buf, ortp_len, ts);
                ot->last_ts = ts;
                if (ret < 0)
                        ortp_delete_target(ot, "send error");
@@ -128,7 +131,7 @@ static void ortp_init_session(struct ortp_target *ot)
        set_multicast(s);
 }
 
-/* called by afs */
+/* called by vss */
 static void ortp_shutdown_targets(void)
 {
        unsigned char buf[ORTP_AUDIO_HEADER_LEN];
@@ -168,14 +171,14 @@ static void ortp_send(long unsigned current_chunk, long unsigned chunks_sent,
 {
        struct ortp_target *ot, *tmp;
        size_t sendbuf_len;
-       int header_len = 0;
+       size_t header_len = 0;
        int packet_type = ORTP_DATA;
        char *sendbuf, *header_buf = NULL;
        struct timeval *chunk_tv;
 
        if (self->status != SENDER_ON)
                return;
-       chunk_tv = afs_chunk_time();
+       chunk_tv = vss_chunk_time();
        if (!chunk_tv)
                return;
        list_for_each_entry_safe(ot, tmp, &targets, node) {
@@ -186,14 +189,14 @@ static void ortp_send(long unsigned current_chunk, long unsigned chunks_sent,
                }
                if (!ot->chunk_ts)
                        ot->chunk_ts = rtp_session_time_to_ts(ot->session,
-                               tv2ms(chunk_tv));
+                               (int)tv2ms(chunk_tv));
 //             PARA_DEBUG_LOG("len: %d, ts: %lu, ts: %d\n",
 //                     len, ot->chunk_ts * chunks_sent, ot->chunk_ts);
                ot->streaming = 1;
        }
        if (list_empty(&targets))
                return;
-       header_buf = afs_get_header(&header_len);
+       header_buf = vss_get_header(&header_len);
        if (!need_extra_header(current_chunk))
                header_len = 0;
        sendbuf_len = ORTP_AUDIO_HEADER_LEN + header_len + len;