On Wed, Jul 01, 2015 at 02:52:47PM +0200, Patrik Jakobsson wrote:
[...]
> --- a/drm.c
> +++ b/drm.c
> @@ -35,6 +35,9 @@
>
> #define DRM_MAX_NAME_LEN 128
>
> +extern int drm_i915_decode_number(struct tcb *tcp, unsigned int arg);
Please rename "arg" to "code", and ...
> +extern int drm_i915_ioctl(struct tcb *tcp, const unsigned int code, long arg);
... move both declarations to defs.h to make them visible also
in the file where these functions are defined.
[...]
> +static int i915_setparam(struct tcb *tcp, const unsigned int code, long arg)
> +{
> + struct drm_i915_setparam param;
> +
> + if (entering(tcp)) {
> + if (umove(tcp, arg, ¶m))
> + return 0;
> +
> + tprints(", {param=");
> + printxval(drm_i915_setparams, param.param, "I915_PARAM_???");
> + tprintf(", value=%d}", param.value);
> + }
> +
> + return 1;
> +}
In this and most of other parsers of _IOC_WRITE ioctls added by this and
the next patches, any error in parser that leads to "return 0" will result
to disabled "arg" decoding, including the fallback decoding performed by
sys_ioctl.
Maybe it's time to deal with this issue in a more generic way.
--
ldv
Attachment:
pgphkpLN4Bbpl.pgp
Description: PGP signature
_______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
