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 | |
| 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')
| -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); |
