wtay gst-plugins-base: gst-plugins-base/ gst-plugins-base/gst-libs/gst/rtsp/

CVS Root:       /cvs/gstreamer
Module:         gst-plugins-base
Changes by:     wtay
Date:           Thu Nov 27 2008  11:16:58 UTC

Log message:
Patch by: 이문형 <iwings at gmail dot com>
* gst-libs/gst/rtsp/gstrtspconnection.c:
(gst_rtsp_connection_connect):
A successful gst_poll_wait() doesn't always mean successful connect() on
Windows.  We should check errors by calling gst_poll_fd_has_error().
See #561924.

Modified files:
    .               : ChangeLog
    gst-libs/gst/rtsp: gstrtspconnection.c

Links:
http://freedesktop.org/cgi-bin/viewcvs.cgi/gstreamer/gst-plugins-base/ChangeLog.diff?r1=1.4218&r2=1.4219
http://freedesktop.org/cgi-bin/viewcvs.cgi/gstreamer/gst-plugins-base/gst-libs/gst/rtsp/gstrtspconnection.c.diff?r1=1.21&r2=1.22

====Begin Diffs====
Index: ChangeLog
===================================================================
RCS file: /cvs/gstreamer/gst-plugins-base/ChangeLog,v
retrieving revision 1.4218
retrieving revision 1.4219
diff -u -d -r1.4218 -r1.4219
--- ChangeLog	25 Nov 2008 15:33:29 -0000	1.4218
+++ ChangeLog	27 Nov 2008 11:16:42 -0000	1.4219
@@ -1,3 +1,13 @@
+2008-11-27  Wim Taymans  <[email protected]>
+
+	Patch by: 이문형 <iwings at gmail dot com>
+	* gst-libs/gst/rtsp/gstrtspconnection.c:
+	(gst_rtsp_connection_connect):
+	A successful gst_poll_wait() doesn't always mean successful connect() on
+	Windows.  We should check errors by calling gst_poll_fd_has_error().
+	See #561924.
 2008-11-25  Wim Taymans  <[email protected]>
 
 	* gst-libs/gst/rtp/gstrtcpbuffer.c:
Index: gstrtspconnection.c
RCS file: /cvs/gstreamer/gst-plugins-base/gst-libs/gst/rtsp/gstrtspconnection.c,v
retrieving revision 1.21
retrieving revision 1.22
diff -u -d -r1.21 -r1.22
--- gstrtspconnection.c	3 Nov 2008 10:49:24 -0000	1.21
+++ gstrtspconnection.c	27 Nov 2008 11:16:44 -0000	1.22
@@ -287,6 +287,10 @@
   else if (retval == -1)
     goto sys_error;
+  /* we can still have an error connecting on windows */
+  if (gst_poll_fd_has_error (conn->fdset, &conn->fd))
+    goto sys_error;
   gst_poll_fd_ignored (conn->fdset, &conn->fd);
 done:

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/

_______________________________________________
gstreamer-cvs-verbose mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/gstreamer-cvs-verbose