summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorOleg Nesterov <oleg@tv-sign.ru>2007-01-23 20:04:13 -0300
committerMauro Carvalho Chehab <mchehab@infradead.org>2007-01-23 22:00:21 -0200
commit419dd8378dfa32985672ab7927b4bc827f33b332 (patch)
tree1dd1dad1ae3593a7042b993fd21cbdee626a9ac8
parent99ddcc7edbb6b1d35a542f6bf0383f1f5a1aa0b0 (diff)
downloadop-kernel-dev-419dd8378dfa32985672ab7927b4bc827f33b332.zip
op-kernel-dev-419dd8378dfa32985672ab7927b4bc827f33b332.tar.gz
V4L/DVB (5123): Buf_qbuf: fix: videobuf_queue->stream corruption and lockup
We are doing ->buf_prepare(buf) before adding buf to q->stream list. This means that videobuf_qbuf() should not try to re-add a STATE_PREPARED buffer. Signed-off-by: Oleg Nesterov <oleg@tv-sign.ru> Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
-rw-r--r--drivers/media/video/video-buf.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/media/video/video-buf.c b/drivers/media/video/video-buf.c
index 635d102..6504a58 100644
--- a/drivers/media/video/video-buf.c
+++ b/drivers/media/video/video-buf.c
@@ -700,6 +700,7 @@ videobuf_qbuf(struct videobuf_queue *q,
goto done;
}
if (buf->state == STATE_QUEUED ||
+ buf->state == STATE_PREPARED ||
buf->state == STATE_ACTIVE) {
dprintk(1,"qbuf: buffer is already queued or active.\n");
goto done;
OpenPOWER on IntegriCloud