Re: [FFmpeg-devel] [PATCH] lavc/v4l2_m2m: don't close the file descriptor we don't own

2019-09-02 Thread Pavel Koshevoy
On Mon, Sep 2, 2019 at 10:40 AM Aman Gupta wrote: > > > > On Mon, Sep 2, 2019 at 12:27 AM Pavel Koshevoy wrote: >> >> ff_v4l2_m2m_create_context initialized V4L2m2mContext.fd to 0 >> which is a valid file descriptor value. Next ff_v4l2_m2m_codec_init >> failed and v4l2_m2m_destroy_context closed

Re: [FFmpeg-devel] [PATCH] lavc/v4l2_m2m: don't close the file descriptor we don't own

2019-09-02 Thread Aman Gupta
On Mon, Sep 2, 2019 at 12:27 AM Pavel Koshevoy wrote: > ff_v4l2_m2m_create_context initialized V4L2m2mContext.fd to 0 > which is a valid file descriptor value. Next ff_v4l2_m2m_codec_init > failed and v4l2_m2m_destroy_context closed file descriptor 0 even > though it didn't belong to V4L2m2mConte

Re: [FFmpeg-devel] [PATCH] lavc/v4l2_m2m: don't close the file descriptor we don't own

2019-09-02 Thread Nicolas George
Pavel Koshevoy (12019-09-02): > ff_v4l2_m2m_create_context initialized V4L2m2mContext.fd to 0 > which is a valid file descriptor value. Next ff_v4l2_m2m_codec_init > failed and v4l2_m2m_destroy_context closed file descriptor 0 even > though it didn't belong to V4L2m2mContext. > --- > libavcodec/v4

[FFmpeg-devel] [PATCH] lavc/v4l2_m2m: don't close the file descriptor we don't own

2019-09-02 Thread Pavel Koshevoy
ff_v4l2_m2m_create_context initialized V4L2m2mContext.fd to 0 which is a valid file descriptor value. Next ff_v4l2_m2m_codec_init failed and v4l2_m2m_destroy_context closed file descriptor 0 even though it didn't belong to V4L2m2mContext. --- libavcodec/v4l2_m2m.c | 1 + 1 file changed, 1 insertio