diff options
author | Mauro Carvalho Chehab <mchehab@redhat.com> | 2011-02-11 08:58:30 -0200 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2011-02-11 09:24:53 -0200 |
commit | 362a97d43c063809f9d64f9c762f50c382fd4cd3 (patch) | |
tree | 666957375033c28c87e0fad02cf32a785d10d260 | |
parent | 2cca68d701d6b803b200ae064a4b77123d9a03f5 (diff) |
Return BadMatch if a Port Attribute is not found
Fix a small non-compliance issue: if a port attribute is not known,
according with Xv specs, it should return BadMatch, and not BadValue.
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
-rw-r--r-- | src/v4l.c | 4 |
1 files changed, 4 insertions, 0 deletions
@@ -790,6 +790,8 @@ V4lSetPortAttribute(ScrnInfoPtr pScrn, /* not mine -> pass to yuv scaler driver */ if (0 != pPPriv->yuv_format && pPPriv->myfmt->setAttribute) ret = pPPriv->myfmt->setAttribute(pScrn, attribute, value); + else + ret = BadMatch; goto err; } if (pPPriv->XvV4LCtrl[i].qctrl.flags & V4L2_CTRL_FLAG_DISABLED) @@ -839,6 +841,8 @@ V4lGetPortAttribute(ScrnInfoPtr pScrn, /* not mine -> pass to yuv scaler driver */ if (0 != pPPriv->yuv_format && pPPriv->myfmt->getAttribute) ret = pPPriv->myfmt->getAttribute(pScrn, attribute, value); + else + ret = BadMatch; goto err; } if (pPPriv->XvV4LCtrl[i].qctrl.flags & V4L2_CTRL_FLAG_DISABLED) |