svn commit: samba r5854 - in branches/SAMBA_3_0/source/smbd: .

[jmcd at samba.org]

Author: jmcd
Date: 2005-03-17 14:32:52 +0000 (Thu, 17 Mar 2005)
New Revision: 5854

WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=5854

Log:
Fix coredump when OS/2 checks for long file name support (with .+,;=[].).
Bugzilla 2338.

Thanks to Guenter Kukkukk <guenter.kukkukk at kukkukk.com>

Modified:
   branches/SAMBA_3_0/source/smbd/open.c


Changeset:
Modified: branches/SAMBA_3_0/source/smbd/open.c
===================================================================
--- branches/SAMBA_3_0/source/smbd/open.c	2005-03-17 12:45:10 UTC (rev 5853)
+++ branches/SAMBA_3_0/source/smbd/open.c	2005-03-17 14:32:52 UTC (rev 5854)
@@ -1146,17 +1146,15 @@
 		oplock_request = 0;
 	}
 
-	/* this is for OS/2 EAs - try and say we don't support them */
+	/* this is for OS/2 long file names - say we don't support them */
 	if (strstr(fname,".+,;=[].")) {
 		unix_ERR_class = ERRDOS;
 		/* OS/2 Workplace shell fix may be main code stream in a later release. */ 
-#if 1 /* OS2_WPS_FIX - Recent versions of OS/2 need this. */
 		unix_ERR_code = ERRcannotopen;
-#else /* OS2_WPS_FIX */
-		unix_ERR_code = ERROR_EAS_NOT_SUPPORTED;
-#endif /* OS2_WPS_FIX */
-
-		DEBUG(5,("open_file_shared: OS/2 EA's are not supported.\n"));
+		unix_ERR_ntstatus = NT_STATUS_OBJECT_NAME_NOT_FOUND;
+		DEBUG(5,("open_file_shared: OS/2 long filenames are not supported.\n"));
+		/* need to reset errno or DEVELOPER will cause us to coredump */
+		errno = 0;
 		file_free(fsp);
 		return NULL;
 	}