On Sat, Mar 14, 2015 at 11:30 AM, Luca Barbato <[email protected]> wrote: > And notify why the capture is impossible. > --- > libavdevice/xcbgrab.c | 13 +++++++++++-- > 1 file changed, 11 insertions(+), 2 deletions(-) > > diff --git a/libavdevice/xcbgrab.c b/libavdevice/xcbgrab.c > index cdc83c4..2a40905 100644 > --- a/libavdevice/xcbgrab.c > +++ b/libavdevice/xcbgrab.c > @@ -527,8 +527,17 @@ static int create_stream(AVFormatContext *s) > gc = xcb_get_geometry(c->conn, c->screen->root); > geo = xcb_get_geometry_reply(c->conn, gc, NULL); > > - c->width = FFMIN(geo->width, c->width); > - c->height = FFMIN(geo->height, c->height); > + if (c->x + c->width >= geo->width || > + c->y + c->height >= geo->height) { > + av_log(s, AV_LOG_ERROR, > + "Capture area %dx%d at position %d.%d " > + "outside the screen size %dx%d\n", > + c->width, c->height, > + c->x, c->y, > + geo->width, geo->height); > + return AVERROR(EINVAL); > + } > + > c->time_base = (AVRational){ st->avg_frame_rate.den, > st->avg_frame_rate.num }; > c->time_frame = av_gettime(); > --
seems ok -- Vittorio _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
