- this patch allows OpenOCD running under eCos to access files via tftp

(thanks to oyvind Harboe for this patch)

git-svn-id: svn://svn.berlios.de/openocd/trunk@242 b42882b7-edfa-0310-969c-e2dbd0fdcd60

2 files changed, 18 insertions, 38 deletions

diff --git a/src/helper/fileio.c b/src/helper/fileio.c
index f98d634f..351c8e04 100644
--- a/src/helper/fileio.c
+++ b/src/helper/fileio.c
@@ -44,23 +44,7 @@ int fileio_open_local(fileio_t *fileio)
 	fileio_local_t *fileio_local = malloc(sizeof(fileio_local_t));
 	char access[4];
 	
-	if ((fileio->access != FILEIO_WRITE) && (fileio->access != FILEIO_READWRITE))
-	{
-		if (stat(fileio->url, &fileio_local->file_stat) == -1)
-		{
-			free(fileio_local);
-			snprintf(fileio->error_str, FILEIO_MAX_ERROR_STRING,
-				"couldn't stat() %s: %s", fileio->url, strerror(errno));
-			return ERROR_FILEIO_NOT_FOUND;
-		}
-	
-		if (S_ISDIR(fileio_local->file_stat.st_mode))
-		{
-			free(fileio_local);
-			snprintf(fileio->error_str, FILEIO_MAX_ERROR_STRING, "%s is a directory", fileio->url);
-			return ERROR_FILEIO_NOT_FOUND;
-		}
-	}
+	fileio->location_private = fileio_local;
 	
 	switch (fileio->access)
 	{
@@ -84,17 +68,6 @@ int fileio_open_local(fileio_t *fileio)
 			ERROR("BUG: access neither read, write nor readwrite");
 			return ERROR_INVALID_ARGUMENTS;
 	}
-	
-	if (fileio->access == FILEIO_READ)
-	{
-		if (fileio_local->file_stat.st_size == 0)
-		{
-			/* tried to open an empty file for reading */
-			free(fileio_local);
-			snprintf(fileio->error_str, FILEIO_MAX_ERROR_STRING, "empty file %s", fileio->url);
-			return ERROR_FILEIO_OPERATION_FAILED;
-		}
-	}
 
 	/* win32 always opens in binary mode */
 #ifndef _WIN32
@@ -111,11 +84,22 @@ int fileio_open_local(fileio_t *fileio)
 		return ERROR_FILEIO_OPERATION_FAILED;
 	}
 	
-	fileio->location_private = fileio_local;
-	
 	if ((fileio->access != FILEIO_WRITE) || (fileio->access == FILEIO_READWRITE))
 	{
-		fileio->size = fileio_local->file_stat.st_size;
+		// NB! Here we use fseek() instead of stat(), since stat is a 
+		// more advanced operation that might not apply to e.g. a disk path
+		// that refers to e.g. a tftp client
+		int result=fseek(fileio_local->file, 0, SEEK_END);
+
+		fileio->size = ftell(fileio_local->file);
+		
+		int result2 = fseek(fileio_local->file, 0, SEEK_SET); 
+			
+		if ((fileio->size<0)||(result<0)||(result2<0))
+		{
+			fileio_close(fileio);
+			return ERROR_FILEIO_OPERATION_FAILED;
+		}
 	}
 	else
 	{
@@ -130,17 +114,14 @@ int fileio_open(fileio_t *fileio, char *url, enum fileio_access access,	enum fil
 	int retval = ERROR_OK;
 	char *resource_identifier = NULL;
 
-	/* try to identify file location */
+	/* try to identify file location. We only hijack the file paths we understand, the rest is
+	 * passed on to the OS which might implement e.g. tftp via a mounted tftp device.
+	 */
 	if ((resource_identifier = strstr(url, "bootp://")) && (resource_identifier == url))
 	{
 		ERROR("bootp resource location isn't supported yet");
 		return ERROR_FILEIO_RESOURCE_TYPE_UNKNOWN;
 	}
-	else if ((resource_identifier = strstr(url, "tftp://")) && (resource_identifier == url))
-	{
-		ERROR("tftp resource location isn't supported yet");
-		return ERROR_FILEIO_RESOURCE_TYPE_UNKNOWN;
-	}
 	else
 	{
 		/* default to local files */
diff --git a/src/helper/fileio.h b/src/helper/fileio.h
index 20e5c629..c67be2c7 100644
--- a/src/helper/fileio.h
+++ b/src/helper/fileio.h
@@ -74,7 +74,6 @@ typedef struct fileio_s
 typedef struct fileio_local_s
 {
 	FILE *file;
-	struct stat file_stat;
 } fileio_local_t;
 
 extern int fileio_write(fileio_t *fileio, u32 size, u8 *buffer, u32 *size_written);