Re: [PATCH 1/2] simplify xfs_vn_listxattr

[Christoph Hellwig <hch@xxxxxx>]

On Fri, Jun 06, 2008 at 02:41:12PM +1000, Timothy Shimmin wrote:
> Need to fix param type.
> 
> re:
> 
> static int
> list_one_attr(const char *name, const size_t len, void *data,
>                 size_t size, ssize_t *result)
> 
> typedef struct xfs_attr_list_context {
>         struct xfs_inode                *dp;            /* inode */
>         struct attrlist_cursor_kern     *cursor;        /* position in list */
>         char                            *alist;         /* output buffer */
>         int                             seen_enough;    /* T/F: seen enough 
> of list? */
>         int                             count;          /* num used entries */
> 
> => mismatch type on &context.count (int) and list_one_attr's result
> param (ssize_t => long)

Yes, didn't see that on my 32-bit machine :)  count should probably
be ssize_t.