<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=gb2312">
<style type="text/css" style="display:none;"><!-- P {margin-top:0;margin-bottom:0;} --></style>
</head>
<body dir="ltr">
<div id="divtagdefaultwrapper" style="color: rgb(0, 0, 0); font-family: Calibri,Helvetica,sans-serif,'EmojiFont','Apple Color Emoji', 'Segoe UI Emoji', NotoColorEmoji, 'Segoe UI Symbol', 'Android Emoji', EmojiSymbols; font-size: 12pt;" dir="ltr">
<p><br>
</p>
<br>
<br>
<div style="color: rgb(0, 0, 0);">
<div>
<hr tabindex="-1" style="width: 98%; display: inline-block;">
<div id="x_divRplyFwdMsg" dir="ltr"><font color="#000000" face="Calibri, sans-serif" style="font-size: 11pt;"><b>发件人:</b> Florian Westphal <fw@strlen.de><br>
<b>发送时间:</b> 2019年2月22日 19:52<br>
<b>收件人:</b> Li,Rongqing<br>
<b>抄送:</b> netfilter-devel@vger.kernel.org; coreteam@netfilter.org; bridge@lists.linux-foundation.org<br>
<b>主题:</b> Re: [PATCH][nf-next] netfilter: convert the proto argument from u8 to u16</font>
<div> </div>
</div>
</div>
<font size="2"><span style="font-size: 10pt;">
<div class="PlainText"><br>
<br>
<br>
>Otherwise, could you add an example where we hit a checkentry failure in<br>
error?</div>
<div class="PlainText"><br>
</div>
<div class="PlainText">true, current kernel has not this case.<br>
<br>
>> +int xt_check_match(struct xt_mtchk_param *, unsigned int size, u_int16_t proto,<br>
>> bool inv_proto);<br>
<br>
>I think you can use u16 instead of the old u_int16_t.<br>
<br>
>Other than that, this looks good to me, thanks.</div>
<div class="PlainText"><br>
</div>
<div class="PlainText">thanks, I will send v2</div>
<div class="PlainText"><br>
</div>
<div class="PlainText">-RongQing<br>
</div>
</span></font></div>
</div>
</body>
</html>