diff options
| author | Dan Carpenter <dan.carpenter@linaro.org> | 2025-03-12 11:33:03 +0300 |
|---|---|---|
| committer | Andreas Gruenbacher <agruenba@redhat.com> | 2025-03-12 13:00:28 +0100 |
| commit | 951d701ef1bbce4c5f88466f5e829fb3d5735856 (patch) | |
| tree | 21f0542848c04ce36aba2d5eb6aa173c9ec7de57 | |
| parent | 0776a508d1d4480d23873ccbee336123c992ff6e (diff) | |
gfs2: Fix a NULL vs IS_ERR() bug in gfs2_find_jhead()
The filemap_grab_folio() function doesn't return NULL, it returns error
pointers. Fix the check to match.
Fixes: 40829760096d ("gfs2: Convert gfs2_find_jhead() to use a folio")
Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com>
| -rw-r--r-- | fs/gfs2/lops.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/gfs2/lops.c b/fs/gfs2/lops.c index 3853c422040b..0fd3b5ec7d8c 100644 --- a/fs/gfs2/lops.c +++ b/fs/gfs2/lops.c @@ -530,8 +530,8 @@ int gfs2_find_jhead(struct gfs2_jdesc *jd, struct gfs2_log_header_host *head, if (!folio) { folio = filemap_grab_folio(mapping, block >> shift); - if (!folio) { - ret = -ENOMEM; + if (IS_ERR(folio)) { + ret = PTR_ERR(folio); done = true; goto out; } |