[CIFS] cleanup sparse and compile errors in previous fix
Signed-off-by: Steve French (sfrench@us.ibm.com)
This commit is contained in:
@@ -907,9 +907,10 @@ static ssize_t cifs_write(struct file *file, const char *write_data,
|
|||||||
return total_written;
|
return total_written;
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct cifsFileInfo *find_writable_file(struct cifsInodeInfo *cifs_inode)
|
struct cifsFileInfo *find_writable_file(struct cifsInodeInfo *cifs_inode)
|
||||||
{
|
{
|
||||||
struct cifsFileInfo *open_file;
|
struct cifsFileInfo *open_file;
|
||||||
|
int rc;
|
||||||
|
|
||||||
read_lock(&GlobalSMBSeslock);
|
read_lock(&GlobalSMBSeslock);
|
||||||
list_for_each_entry(open_file, &cifs_inode->openFileList, flist) {
|
list_for_each_entry(open_file, &cifs_inode->openFileList, flist) {
|
||||||
@@ -920,7 +921,7 @@ static struct cifsFileInfo *find_writable_file(struct cifsInodeInfo *cifs_inode)
|
|||||||
(open_file->pfile->f_flags & O_WRONLY))) {
|
(open_file->pfile->f_flags & O_WRONLY))) {
|
||||||
read_unlock(&GlobalSMBSeslock);
|
read_unlock(&GlobalSMBSeslock);
|
||||||
if(open_file->invalidHandle) {
|
if(open_file->invalidHandle) {
|
||||||
rc = cifs_reopen_file(cifs_inode->vfs_inode,
|
rc = cifs_reopen_file(&cifs_inode->vfs_inode,
|
||||||
open_file->pfile, FALSE);
|
open_file->pfile, FALSE);
|
||||||
/* if it fails, try another handle - might be */
|
/* if it fails, try another handle - might be */
|
||||||
/* dangerous to hold up writepages with retry */
|
/* dangerous to hold up writepages with retry */
|
||||||
|
@@ -962,7 +962,6 @@ int cifs_setattr(struct dentry *direntry, struct iattr *attrs)
|
|||||||
struct cifsTconInfo *pTcon;
|
struct cifsTconInfo *pTcon;
|
||||||
char *full_path = NULL;
|
char *full_path = NULL;
|
||||||
int rc = -EACCES;
|
int rc = -EACCES;
|
||||||
int found = FALSE;
|
|
||||||
struct cifsFileInfo *open_file = NULL;
|
struct cifsFileInfo *open_file = NULL;
|
||||||
FILE_BASIC_INFO time_buf;
|
FILE_BASIC_INFO time_buf;
|
||||||
int set_time = FALSE;
|
int set_time = FALSE;
|
||||||
@@ -970,7 +969,6 @@ int cifs_setattr(struct dentry *direntry, struct iattr *attrs)
|
|||||||
__u64 uid = 0xFFFFFFFFFFFFFFFFULL;
|
__u64 uid = 0xFFFFFFFFFFFFFFFFULL;
|
||||||
__u64 gid = 0xFFFFFFFFFFFFFFFFULL;
|
__u64 gid = 0xFFFFFFFFFFFFFFFFULL;
|
||||||
struct cifsInodeInfo *cifsInode;
|
struct cifsInodeInfo *cifsInode;
|
||||||
struct list_head *tmp;
|
|
||||||
|
|
||||||
xid = GetXid();
|
xid = GetXid();
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user