Re: [PATCH 04/19] mkfs: validate all input values

[Jan Tulak <jtulak@xxxxxxxxxx>]

On Thu, Apr 7, 2016 at 1:02 AM, Eric Sandeen <sandeen@xxxxxxxxxxx> wrote:

On 3/24/16 6:15 AM, jtulak@xxxxxxxxxx wrote:
> From: Dave Chinner <dchinner@xxxxxxxxxx>
>
> CHANGELOG
> o Fix an unsigned < 0 comparison - save getnum result to another variable
>Â Âand test it, before converting it to unsigned logagno.

...

> @@ -1468,7 +1485,10 @@ main(
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Ârespec('l', lopts, L_AGNUM);
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (ldflag)
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âconflict('l', lopts, L_AGNUM, L_DEV);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âlogagno = atoi(value);
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âtmp_num = getnum(value, 0, 0, false);
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (tmp_num < 0)
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âillegal(value, "l agno");
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âlogagno = (xfs_agnumber_t)tmp_num;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âlaflag = 1;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbreak;
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcase L_FILE:

Why not cast to (__int64_t) like other variables do? i.e.


+Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âlogagno = getnum(value, 0, 0, false);
+Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif ((__int64_t)logagno < 0)
+Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âillegal(value, "l agno");

or am I missing something?

Nothing I'm aware of. Replaced.

Â

Thanks,
-Eric

_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs

--

Jan Tulak

jtulak@xxxxxxxxxxÂ/ jan@xxxxxxxx