Fixed potential memory leak, now always calling WaveConfirm after playing
authorArmin Novak <anovak@thinstuff.com>
Fri, 20 Sep 2013 07:19:11 +0000 (09:19 +0200)
committerArmin Novak <anovak@thinstuff.com>
Fri, 20 Sep 2013 07:19:11 +0000 (09:19 +0200)
a sample.

channels/rdpsnd/client/alsa/rdpsnd_alsa.c
channels/rdpsnd/client/rdpsnd_main.c

index e775e48..bfb995c 100644 (file)
@@ -613,8 +613,6 @@ static void rdpsnd_alsa_wave_play(rdpsndDevicePlugin* device, RDPSND_WAVE* wave)
        wave->wTimeStampB = wave->wTimeStampA + wave->wLatency;
 
        //fprintf(stderr, "wTimeStampA: %d wTimeStampB: %d wLatency: %d\n", wave->wTimeStampA, wave->wTimeStampB, wave->wLatency);
-
-       device->WaveConfirm(device, wave);
 }
 
 static COMMAND_LINE_ARGUMENT_A rdpsnd_alsa_args[] =
index 5047ed7..613f160 100644 (file)
@@ -442,8 +442,8 @@ static void rdpsnd_recv_wave_pdu(rdpsndPlugin* rdpsnd, wStream* s)
        {
                wave->wTimeStampB = rdpsnd->wTimeStamp + wave->wAudioLength + TIME_DELAY_MS;
                wave->wLocalTimeB = wave->wLocalTimeA + wave->wAudioLength + TIME_DELAY_MS;
-               rdpsnd->device->WaveConfirm(rdpsnd->device, wave);
        }
+       rdpsnd->device->WaveConfirm(rdpsnd->device, wave);
 }
 
 static void rdpsnd_recv_close_pdu(rdpsndPlugin* rdpsnd)