X-Git-Url: http://www.aleph1.co.uk/gitweb/?p=yaffs2.git;a=blobdiff_plain;f=direct%2Fyaffsfs.c;h=3ee5f0c8644a9449b6ae3776533bd747c7cee6b5;hp=0b236ff86669f0545e38ef0df85bb807c2b1a5f5;hb=62003e4e28d993ae2182cb45276bc305ee656ce4;hpb=3fb0edc9f04818ed9f3ffe966d125a0e442843c2

diff --git a/direct/yaffsfs.c b/direct/yaffsfs.c
index 0b236ff..3ee5f0c 100644
--- a/direct/yaffsfs.c
+++ b/direct/yaffsfs.c
@@ -612,7 +612,10 @@ int yaffs_open_sharing(const YCHAR *path, int oflag, int mode, int sharing)
 
 	handle = yaffsfs_GetNewHandle();
 
-	if(handle >= 0){
+	if(handle < 0){
+		yaffsfs_SetError(-ENFILE);
+		errorReported = 1;
+	} else {
 
 		yh = yaffsfs_GetHandlePointer(handle);
 
@@ -1055,30 +1058,28 @@ off_t yaffs_lseek(int fd, off_t offset, int whence)
 	h = yaffsfs_GetHandlePointer(fd);
 	obj = yaffsfs_GetHandleObject(fd);
 
-	if(!h || !obj)
+	if(!h || !obj){
 		/* bad handle */
 		yaffsfs_SetError(-EBADF);
-	else if(whence == SEEK_SET){
-		if(offset >= 0)
-			pos = offset;
-	}
-	else if(whence == SEEK_CUR) {
-		if( (h->position + offset) >= 0)
-			pos = (h->position + offset);
-	}
-	else if(whence == SEEK_END) {
-		fSize = yaffs_get_obj_length(obj);
-		if(fSize >= 0 && (fSize + offset) >= 0)
-			pos = fSize + offset;
-	}
-
-	if(pos >= 0)
-		h->position = pos;
-	else {
-		/* todo error */
+	} else {
+		if(whence == SEEK_SET){
+			if(offset >= 0)
+				pos = offset;
+		} else if(whence == SEEK_CUR) {
+			if( (h->position + offset) >= 0)
+				pos = (h->position + offset);
+		} else if(whence == SEEK_END) {
+			fSize = yaffs_get_obj_length(obj);
+			if(fSize >= 0 && (fSize + offset) >= 0)
+				pos = fSize + offset;
+		} 
+
+		if(pos >= 0)
+			h->position = pos;
+		else
+			yaffsfs_SetError(-EINVAL);
 	}
 
-
 	yaffsfs_Unlock();
 
 	return pos;
@@ -1727,8 +1728,40 @@ int yaffs_mkdir(const YCHAR *path, mode_t mode)
 	yaffs_obj_t *parent = NULL;
 	yaffs_obj_t *dir = NULL;
 	YCHAR *name;
+	YCHAR *use_path = NULL;
+	int path_length = 0;
 	int retVal= -1;
+	int i;
 
+
+	/*
+	 * We don't have a definition for max path length.
+	 * We will use 3 * max name length instead.
+	 */
+	
+	path_length = strnlen(path,(YAFFS_MAX_NAME_LENGTH+1)*3 +1);
+
+	/* If the last character is a path divider, then we need to
+	 * trim it back so that the name look-up works properly.
+	 * eg. /foo/new_dir/ -> /foo/newdir
+	 * Curveball: Need to handle multiple path dividers:
+	 * eg. /foof/sdfse///// -> /foo/sdfse
+	 */
+	if(path_length > 0 && 
+		yaffsfs_IsPathDivider(path[path_length-1])){
+		use_path = YMALLOC(path_length + 1);
+		if(!use_path){
+			yaffsfs_SetError(-ENOMEM);
+			return -1;
+		}
+		strcpy(use_path, path);
+		for(i = path_length-1;
+			i >= 0 && yaffsfs_IsPathDivider(use_path[i]);
+			i--)
+			use_path[i] = (YCHAR) 0;
+		path = use_path;
+	}
+	
 	yaffsfs_Lock();
 	parent = yaffsfs_FindDirectory(NULL,path,&name,0);
 	if(parent && yaffs_strnlen(name,5) == 0){
@@ -1754,6 +1787,9 @@ int yaffs_mkdir(const YCHAR *path, mode_t mode)
 
 	yaffsfs_Unlock();
 
+	if(use_path)
+		YFREE(use_path);
+
 	return retVal;
 }
 
@@ -2330,6 +2366,11 @@ int yaffs_n_handles(const YCHAR *path)
 	return yaffsfs_CountHandles(obj);
 }
 
+int yaffs_get_error(void)
+{
+	return yaffsfs_GetLastError();
+}
+
 int yaffs_dump_dev(const YCHAR *path)
 {
 #if 0