> On May 9, 2017, at 4:18 PM, Christoph Hellwig <hch@xxxxxx> wrote:
>
> On Tue, May 09, 2017 at 10:40:31AM -0500, Eric Sandeen wrote:
>>> -
>>> - hsize = XFS_HSIZE(handle);
>>> + hsize = sizeof(xfs_handle_t);
>>
>> at this point, no need to have the hsize assignment in if { } as well as else { }, right?
>>
>
> In the other branch hsize is smaller, it only covers the fsid.
Ugh I misread, sorry.
Reviewed-by: Eric Sandeen <sandeen@xxxxxxxxxxx>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
