diff options
author | Lachlan McIlroy <lachlan@redback.melbourne.sgi.com> | 2008-12-04 21:16:15 -0500 |
---|---|---|
committer | Lachlan McIlroy <lachlan@redback.melbourne.sgi.com> | 2008-12-04 21:16:15 -0500 |
commit | c6422617a1c0d7787e515748b01f594fe43aea98 (patch) | |
tree | c9ca43adf13df72cbc4cf861da8c3b0439bc602a /fs/xfs | |
parent | 6a0775a991d5597ce98f1e15373288ea133cc793 (diff) |
[XFS] Check return value of xfs_buf_get_noaddr()
We check the return value of all other calls to xfs_buf_get_noaddr().
Make sense to do it here too.
Signed-off-by: Lachlan McIlroy <lachlan@sgi.com>
Reviewed-by: Christoph Hellwig <hch@infradead.org>
Reviewed-by: Eric Sandeen <sandeen@sandeen.net>
Diffstat (limited to 'fs/xfs')
-rw-r--r-- | fs/xfs/xfs_vnodeops.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/xfs/xfs_vnodeops.c b/fs/xfs/xfs_vnodeops.c index 2d57aae0e310..4547608b46c4 100644 --- a/fs/xfs/xfs_vnodeops.c +++ b/fs/xfs/xfs_vnodeops.c | |||
@@ -3034,6 +3034,8 @@ xfs_zero_remaining_bytes( | |||
3034 | bp = xfs_buf_get_noaddr(mp->m_sb.sb_blocksize, | 3034 | bp = xfs_buf_get_noaddr(mp->m_sb.sb_blocksize, |
3035 | XFS_IS_REALTIME_INODE(ip) ? | 3035 | XFS_IS_REALTIME_INODE(ip) ? |
3036 | mp->m_rtdev_targp : mp->m_ddev_targp); | 3036 | mp->m_rtdev_targp : mp->m_ddev_targp); |
3037 | if (!bp) | ||
3038 | return XFS_ERROR(ENOMEM); | ||
3037 | 3039 | ||
3038 | for (offset = startoff; offset <= endoff; offset = lastoffset + 1) { | 3040 | for (offset = startoff; offset <= endoff; offset = lastoffset + 1) { |
3039 | offset_fsb = XFS_B_TO_FSBT(mp, offset); | 3041 | offset_fsb = XFS_B_TO_FSBT(mp, offset); |