Tim Gardner
Wed, 22 Feb 2012 15:46:16 -0800
On 02/21/2012 10:33 AM, Greg KH wrote:
On Fri, Feb 17, 2012 at 06:49:52AM -0700, Tim Gardner wrote:From: Andy Whitcroft<a...@canonical.com> read() calls against a file descriptor connected to a directory are incorrectly returning EINVAL rather than EISDIR: [EISDIR] [XSI] [Option Start] The fildes argument refers to a directory and the implementation does not allow the directory to be read using read() or pread(). The readdir() function should be used instead. [Option End] This occurs because we do not have a .read operation defined for ecryptfs directories. Connect this up to generic_read_dir(). BugLink: http://bugs.launchpad.net/bugs/719691 Signed-off-by: Andy Whitcroft<a...@canonical.com> Signed-off-by: Tyler Hicks<tyhi...@linux.vnet.ibm.com> (cherry picked from commit 323ef68faf1bbd9b1e66aea268fd09d358d7e8ab) Cc: Tyler Hicks<tyler.hi...@canonical.com> Signed-off-by: Colin Ian King<colin.k...@canonical.com> Signed-off-by: Tim Gardner<tim.gard...@canonical.com>Why did you drop Andy's original signed-off-by for this patch? And Tyler's? Again, not acceptable at all, if you are going to make me verify every patch you send, as being correct, I'm just not going to anymore, as it's a waste of my time, don't you agree? I'll apply Andy's original patch here, as I already did the research for it, but that's it, if you want anything else applied, please resend it _AFTER_ verifying that you didn't strip any credits out of it. greg k-h
I'm not grok'ing what you think is wrong. Andy's s-o-b looks like its there to me. Everything below the 'cherry picked' line references everyone that participated in the bug fix. I've noted Tyler's new email address, Colin selected this patch for application and testing, and I was the final sign-off for the Ubuntu kernel.
rtg -- Tim Gardner tim.gard...@canonical.com -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html