Commit f5c54804d9e3bb23d8924af09d9ca1c8de9560b6

Authored by Steven Whitehouse
1 parent 52ae7b7935

[GFS2] Fix uninitialised variable

This fixes a bug where, in certain cases an uninitialised variable
could cause a dereference of a NULL pointer in gfs2_commit_write().
Also a typo in a comment is fixed at the same time.

Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>

Showing 2 changed files with 2 additions and 1 deletions Side-by-side Diff

fs/gfs2/ops_address.c
... ... @@ -385,6 +385,7 @@
385 385 goto out_unlock;
386 386  
387 387  
  388 + ip->i_alloc.al_requested = 0;
388 389 if (alloc_required) {
389 390 al = gfs2_alloc_get(ip);
390 391  
... ... @@ -32,7 +32,7 @@
32 32 struct gfs2_alloc *gfs2_alloc_get(struct gfs2_inode *ip);
33 33 static inline void gfs2_alloc_put(struct gfs2_inode *ip)
34 34 {
35   - return; /* Se we can see where ip->i_alloc is used */
  35 + return; /* So we can see where ip->i_alloc is used */
36 36 }
37 37  
38 38 int gfs2_inplace_reserve_i(struct gfs2_inode *ip,