[Redbutton-devel] SF.net SVN: redbutton: [11] redbutton-download/trunk

[<ski...@us...>]

Revision: 11
Author:   skilvington
Date:     2006-03-02 09:11:17 -0800 (Thu, 02 Mar 2006)
ViewCVS:  http://svn.sourceforge.net/redbutton/?rev=11&view=rev

Log Message:
-----------
retry reading demux device after EOVERFLOW errors

Modified Paths:
--------------
    redbutton-download/trunk/TODO
    redbutton-download/trunk/table.c
Modified: redbutton-download/trunk/TODO
===================================================================
--- redbutton-download/trunk/TODO	2006-03-02 17:10:15 UTC (rev 10)
+++ redbutton-download/trunk/TODO	2006-03-02 17:11:17 UTC (rev 11)
@@ -1,7 +1,3 @@
-work out why this happened on BBC1 once:
-read: Value too large for defined data type
-Unable to read PID
-
 have a verbose flag
 
 use a linked list of modules rather than an array in struct carousel
@@ -11,9 +7,6 @@
 => need to download again if it gets bigger
 (we currently just look at the module version, is this enough?)
 
-what happens when you change channel?
-(kill -9 and restart with the new programme_number?)
-
 create carousels/PID/CID dir in add_dsmcc_pid()
 (may need to use carousels/<assoc_tag> as stream2pid may return 0)
 

Modified: redbutton-download/trunk/table.c
===================================================================
--- redbutton-download/trunk/table.c	2006-03-02 17:10:15 UTC (rev 10)
+++ redbutton-download/trunk/table.c	2006-03-02 17:11:17 UTC (rev 11)
@@ -169,8 +169,12 @@
 		/* read the table */
 		if((n = read(fd, _buf, sizeof(_buf))) < 0)
 		{
+			/*
+			 * may get EOVERFLOW if we don't read quick enough,
+			 * so just report it and have another go
+			 */
 			error("read: %s", strerror(errno));
-			return NULL;
+			_buf[0] = 0;
 		}
 	}
 	while(_buf[0] != TID_DSMCC_CONTROL && _buf[0] != TID_DSMCC_DATA);



This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.