<div dir="ltr"><div>On Tue, Jun 20, 2017 at 12:47:39PM +0300, Dmitry V. Levin wrote:</div><div>> On Tue, Jun 20, 2017 at 01:11:51PM +0800, JingPiao Chen wrote:</div><div>> > On Tue, Jun 20, 2017 at 03:21:14AM +0300, Dmitry V. Levin wrote:</div><div>> > > On Mon, Jun 19, 2017 at 09:15:27AM +0800, JingPiao Chen wrote:</div><div>> > > > * tests/sock_diag_unix_attrs.c: New file.</div><div>> > > > * tests/<a href="http://gen_tests.in">gen_tests.in</a> (sock_diag_unix_attrs): New entry.</div><div>> > > > * tests/pure_executables.list: Add sock_diag_unix_attrs.</div><div>> > > > * tests/.gitignore: Likewise.</div><div>> > > </div><div>> > > I don't see what's being checked by this test in addition to the previous</div><div>> > > one, but anyway, I suggest a shorter name: nlattr_sock_diag.</div><div>> > </div><div>> > it test:</div><div>> > netlink_sock_diag.c: decode_unix_diag_msg():</div><div>> > </div><div>> > <span class="gmail-Apple-tab-span" style="white-space:pre">      </span>offset = NLMSG_ALIGN(sizeof(msg));                                      </div><div>> > <span class="gmail-Apple-tab-span" style="white-space:pre">  </span>if (decode_nla && len > offset) {                                       </div><div>> > <span class="gmail-Apple-tab-span" style="white-space:pre">              </span>tprints(", ");                                                  </div><div>> > <span class="gmail-Apple-tab-span" style="white-space:pre">          </span>decode_nlattr(tcp, addr + offset, len - offset,</div><div>> > <span class="gmail-Apple-tab-span" style="white-space:pre">                      </span>      unix_diag_attrs, "UNIX_DIAG_???",</div><div>> > <span class="gmail-Apple-tab-span" style="white-space:pre">                 </span>      NULL, 0, NULL);                                   </div><div>> > <span class="gmail-Apple-tab-span" style="white-space:pre">        </span>}</div><div>> > </div><div>> > But when I add netlink_attribute test, this test it useless.</div><div>> > I remove the similar tests ([PATCH v3 05/16], [PATCH v3 07/16],</div><div>> > [PATCH v3 09/16], [PATCH v3 10/16], [PATCH v3 10/16]).</div><div>> > Because will test when attribute data decode are added.</div><div>> </div><div>> Other tests certainly tested something: in v4 at least the following</div><div>> functions are not fully covered by tests:</div><div>> decode_netlink_diag_msg</div><div>> decode_packet_diag_msg</div><div>> decode_inet_diag_req_compat</div><div>> decode_inet_diag_req_v2</div><div><br></div><div>Yes, if you want to test them now, I can send attribute data decode patch.</div><div>What do you think?</div><div><br></div><div>--</div><div>JingPiao Chen</div></div>