diff options
author | Abhi Das <adas@redhat.com> | 2015-05-05 11:26:04 -0500 |
---|---|---|
committer | Bob Peterson <rpeterso@redhat.com> | 2015-05-05 11:26:04 -0500 |
commit | 959b6717175713259664950f3bba2418b038f69a (patch) | |
tree | eb2bc7124b18d8ae3b071b0d8c85a9e0c3a82e94 /fs/gfs2/xattr.c | |
parent | 86fbca4923f956dae31247e68dc73ffdfd6e5cb0 (diff) |
gfs2: handle NULL rgd in set_rgrp_preferences
The function set_rgrp_preferences() does not handle the (rarely
returned) NULL value from gfs2_rgrpd_get_next() and this patch
fixes that.
The fs image in question is only 150MB in size which allows for
only 1 rgrp to be created. The in-memory rb tree has only 1 node
and when gfs2_rgrpd_get_next() is called on this sole rgrp, it
returns NULL. (Default behavior is to wrap around the rb tree and
return the first node to give the illusion of a circular linked
list. In the case of only 1 rgrp, we can't have
gfs2_rgrpd_get_next() return the same rgrp (first, last, next all
point to the same rgrp)... that would cause unintended consequences
and infinite loops.)
Signed-off-by: Abhi Das <adas@redhat.com>
Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Diffstat (limited to 'fs/gfs2/xattr.c')
0 files changed, 0 insertions, 0 deletions