<div id="geary-body" dir="auto"><div><br></div></div><div id="geary-quote" dir="auto"><div><br></div>Hi Laurent,<div><br>On Wed, Apr 15, 2020 at 00:56, Laurent Pinchart <laurent.pinchart@ideasonboard.com> wrote:<br><blockquote type="cite"><div class="plaintext" style="white-space: pre-wrap;">Hi Umang,

Thank you for the patch.

On Tue, Apr 14, 2020 at 07:06:59AM +0000, Umang Jain wrote:
<blockquote> Failing the test guards against negative index (-ENOENT in this case)
 being passed to V4L2BufferCache::put().
</blockquote>
Can this happens in practice ?</div></blockquote><span style="white-space: pre-wrap;"><div><span style="white-space: pre-wrap;"><br></span></div>Well, V4L2BufferCache::put() accepts only unsigned int argument, so I would say, it won't happen in practice.</span></div><div><span style="white-space: pre-wrap;">Still the patches seems correct to me but commit message should be tweaked, is that right?</span></div><div><span style="white-space: pre-wrap;"><br></span></div></div><img src="https://u15657259.ct.sendgrid.net/wf/open?upn=GCEip0g28ftA9O9fsCR2M7x08El53O4YVYtHuSI-2FrLwtytoSlmO-2FnSq-2B0q807R-2FEhV2DhK89qN4OrpehE55o5961aCrpnvgqBeOECmLcrKuqRKOnAFtpdA3JYJS7ikWSCHetcqYMcdKpGGBVyo4S8wPOvgBxqjlcQQmH-2Bshhc6bnpVlbHI7vruErTitncZhg-2BjmOkqt3y-2BVFwDtOlsn4QOs0-2Fx7BkVT6FYidxp6Ws7lga3jA0aYxb6o3Rg-2BOI6ZZxfwk2ST8YIzjey1LubXA8g-3D-3D" alt="" width="1" height="1" border="0" style="height:1px !important;width:1px !important;border-width:0 !important;margin-top:0 !important;margin-bottom:0 !important;margin-right:0 !important;margin-left:0 !important;padding-top:0 !important;padding-bottom:0 !important;padding-right:0 !important;padding-left:0 !important;"/>