diff options
| author | Abhijith Das <adas@redhat.com> | 2012-12-19 10:48:01 -0500 |
|---|---|---|
| committer | Steven Whitehouse <swhiteho@redhat.com> | 2013-01-02 10:04:53 +0000 |
| commit | f1213cacc7ffc7d4cdef3692f22b28a2df3216f5 (patch) | |
| tree | 323a502b74dc45a726b05c5b0bddccb77ba516df /fs/gfs2/recovery.h | |
| parent | ec1487528bed94c4aaff3687834fe94203880fd6 (diff) | |
GFS2: Fix race in gfs2_rs_alloc
QE aio tests uncovered a race condition in gfs2_rs_alloc where it's possible
to come out of the function with a valid ip->i_res allocation but it gets
freed before use resulting in a NULL ptr dereference.
This patch envelopes the initial short-circuit check for non-NULL ip->i_res
into the mutex lock. With this patch, I was able to successfully run the
reproducer test multiple times.
Resolves: rhbz#878476
Signed-off-by: Abhi Das <adas@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/gfs2/recovery.h')
0 files changed, 0 insertions, 0 deletions