[Ocfs2-devel] Maybe a null point bug in __ocfs2_change_file_space.

[Ocfs2-devel] Maybe a null point bug in __ocfs2_change_file_space.

[Chang Limin]

Hi,
 Version linux-3.3
In function
static long ocfs2_fallocate(struct file *file, int mode, loff_t offset,
                        loff_t len)
{
?
return __ocfs2_change_file_space(NULL, inode, offset, cmd, &sr,           // file is NULL
                                        change_size);
}

In function
static int __ocfs2_change_file_space(struct file *file, struct inode *inode,
                                  loff_t f_pos, unsigned int cmd,
                                  struct ocfs2_space_resv *sr,
                                  int change_size)
{
if (file->f_flags & O_SYNC)                                                                                   // access file->f_flags result null pointer
           handle->h_sync = 1;
}

Changlimin

-------------------------------------------------------------------------------------------------------------------------------------
????????????????????????????????????????
????????????????????????????????????????
????????????????????????????????????????
???
This e-mail and its attachments contain confidential information from H3C, which is
intended only for the person or entity whose address is listed above. Any use of the
information contained herein in any way (including, but not limited to, total or partial
disclosure, reproduction, or dissemination) by persons other than the intended
recipient(s) is prohibited. If you receive this e-mail in error, please notify the sender
by phone or email immediately and delete it!
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://oss.oracle.com/pipermail/ocfs2-devel/attachments/20120413/8172e23a/attachment-0001.html 

[Ocfs2-devel] Maybe a null point bug in __ocfs2_change_file_space.

[Tao Ma]

Hi Limin,
	Thanks for the report and the patch is already there. Please search the
archive with the subject "ocfs2: Fix oops in fallocate()".

Thanks
Tao
On 04/13/2012 03:58 PM, Chang Limin wrote:
> Hi,
> 
>  Version linux-3.3
> 
> In function
> 
> static long ocfs2_fallocate(struct file *file, int mode, loff_t offset,
> 
>                         loff_t len)
> 
> {
> 
> ?
> 
> return __ocfs2_change_file_space(NULL, inode, offset, cmd,
> &sr,           // file is NULL
> 
>                                         change_size);
> 
> }
> 
>  
> 
> In function
> 
> static int __ocfs2_change_file_space(struct file *file, struct inode *inode,
> 
>                                   loff_t f_pos, unsigned int cmd,
> 
>                                   struct ocfs2_space_resv *sr,
> 
>                                   int change_size)
> 
> {
> 
> if (file->f_flags &
> O_SYNC)                                                                                  
> // access file->f_flags result null pointer
> 
>            handle->h_sync = 1;
> 
> }
> 
>  
> 
> Changlimin

[Ocfs2-devel] Maybe a null point bug in __ocfs2_change_file_space.

[Chang Limin]

Hi,
 Version linux-3.3
In function
static long ocfs2_fallocate(struct file *file, int mode, loff_t offset,
                        loff_t len)
{
?
return __ocfs2_change_file_space(NULL, inode, offset, cmd, &sr,           // file is NULL
                                        change_size);
}

In function
static int __ocfs2_change_file_space(struct file *file, struct inode *inode,
                                  loff_t f_pos, unsigned int cmd,
                                  struct ocfs2_space_resv *sr,
                                  int change_size)
{
if (file->f_flags & O_SYNC)                                                                                   // access file->f_flags result null pointer
           handle->h_sync = 1;
}

Changlimin
-------------------------------------------------------------------------------------------------------------------------------------
????????????????????????????????????????
????????????????????????????????????????
????????????????????????????????????????
???
This e-mail and its attachments contain confidential information from H3C, which is
intended only for the person or entity whose address is listed above. Any use of the
information contained herein in any way (including, but not limited to, total or partial
disclosure, reproduction, or dissemination) by persons other than the intended
recipient(s) is prohibited. If you receive this e-mail in error, please notify the sender
by phone or email immediately and delete it!
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://oss.oracle.com/pipermail/ocfs2-devel/attachments/20120413/bda432dc/attachment.html