From owner-freebsd-net@FreeBSD.ORG Wed Mar 2 22:57:20 2011 Return-Path: Delivered-To: freebsd-net@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 22FA11065687; Wed, 2 Mar 2011 22:57:20 +0000 (UTC) (envelope-from jamesbrandongooch@gmail.com) Received: from mail-wy0-f182.google.com (mail-wy0-f182.google.com [74.125.82.182]) by mx1.freebsd.org (Postfix) with ESMTP id 566EB8FC12; Wed, 2 Mar 2011 22:57:19 +0000 (UTC) Received: by wyb32 with SMTP id 32so627710wyb.13 for ; Wed, 02 Mar 2011 14:57:18 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:in-reply-to:references:date :message-id:subject:from:to:cc:content-type; bh=WdJ0ZReYMTMSw+qy6sPS791E6H6QDwiU6UtBbTCGFH4=; b=PvOPmdeZ/k5XKct1PI2Rt07kXXNTHNZ/MF7tPMsSzYGJPdJZGi/maKpI5/rSxfsOJ8 4+kdhVeqb2FUUeY5chGeEZ20Cpn8BbR2EAWHkn09A62RAW3PWWhxjBO41Ac1PbFcgULT bsE1LjPVoYWn02ycxkrpUKZYUQTBbbN8WZfvk= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; b=pvVokbN/l26KYca3BI06p7BTML5lnBVaog1lAaxPrW8dokWNY0hj5y2pj9eDFu8GZt 5OG0L9QGckb4AVml5h3uakbdNSmB5wbGMVTqsSJ7Nl8meR5PebsMEdCQh+3NixtjBxrm 7EiNiXcHk4cpgFil6qDGcUHxjvTcUUkGlfC1c= MIME-Version: 1.0 Received: by 10.216.154.136 with SMTP id h8mr225276wek.84.1299106638309; Wed, 02 Mar 2011 14:57:18 -0800 (PST) Received: by 10.216.25.72 with HTTP; Wed, 2 Mar 2011 14:57:18 -0800 (PST) In-Reply-To: <20110302215258.GA48643@atarininja.org> References: <1298961441.2888.2.camel@core.nessbank> <20110302194540.0000018f@unknown> <20110302215258.GA48643@atarininja.org> Date: Wed, 2 Mar 2011 16:57:18 -0600 Message-ID: From: Brandon Gooch To: Wesley Shields Content-Type: multipart/mixed; boundary=0016e649c770b3a94d049d87d503 Cc: Bruce Cran , freebsd-net@freebsd.org, Rui Paulo Subject: Re: Interface descriptions via pcap broken X-BeenThere: freebsd-net@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Networking and TCP/IP with FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 02 Mar 2011 22:57:20 -0000 --0016e649c770b3a94d049d87d503 Content-Type: text/plain; charset=ISO-8859-1 On Wed, Mar 2, 2011 at 3:52 PM, Wesley Shields wrote: > On Wed, Mar 02, 2011 at 07:45:40PM +0000, Bruce Cran wrote: >> On Wed, 2 Mar 2011 13:40:57 -0600 >> Brandon Gooch wrote: >> >> > BTW, if you give your devices descriptions, libpcap >> > starts to behave again; this could be a clue to the source of the bug >> > :/ >> >> 0xa5 is the malloc fill pattern when debugging is enabled, and >> having recently reinstalled FreeBSD I forgot to create the malloc.conf >> symlink. So something's reading from uninitialized memory. > > Does commit c65292b04b98d6a76d58c5a54ca8f81463bf24de in the libpcap git > tree look like it could help? I haven't checked in detail but I think it > might have never made it into a release yet? > > https://github.com/mcr/libpcap/commit/c65292b04b98d6a76d58c5a54ca8f81463bf24de > > -- WXS I applied the patch attached (diff'd against the code from the github link above), rebuilt libpcap and wireshark, and the problem described no longer exists. Thanks for looking at this wxs@! -Brandon --0016e649c770b3a94d049d87d503 Content-Type: application/octet-stream; name="libpcap-inet.c.diff" Content-Disposition: attachment; filename="libpcap-inet.c.diff" Content-Transfer-Encoding: base64 X-Attachment-Id: f_gksuho9c0 LS0tIGNvbnRyaWIvbGlicGNhcC9pbmV0LmMub3JpZwkyMDExLTAyLTAxIDIzOjU3OjMyLjAwMDAw MDAwMCAtMDYwMAorKysgY29udHJpYi9saWJwY2FwL2luZXQuYwkyMDExLTAzLTAyIDE2OjI0OjM0 LjAwMDAwMDAwMCAtMDYwMApAQCAtNDMxLDI2ICs0MzEsNTQgQEAKIAlzdHJsY3B5KGlmcmRlc2Mu aWZyX25hbWUsIG5hbWUsIHNpemVvZiBpZnJkZXNjLmlmcl9uYW1lKTsKIAlzID0gc29ja2V0KEFG X0lORVQsIFNPQ0tfREdSQU0sIDApOwogCWlmIChzID49IDApIHsKKyNpZmRlZiBfX0ZyZWVCU0Rf XworCQkvKgorCQkgKiBPbiBGcmVlQlNELCBpZiB0aGUgYnVmZmVyIGlzbid0IGJpZyBlbm91Z2gg Zm9yIHRoZQorCQkgKiBkZXNjcmlwdGlvbiwgdGhlIGlvY3RsIHN1Y2NlZWRzLCBidXQgdGhlIGRl c2NyaXB0aW9uCisJCSAqIGlzbid0IGNvcGllZCwgaWZyX2J1ZmZlci5sZW5ndGggaXMgc2V0IHRv IHRoZSBkZXNjcmlwdGlvbgorCQkgKiBsZW5ndGgsIGFuZCBpZnJfYnVmZmVyLmJ1ZmZlciBpcyBz ZXQgdG8gTlVMTC4KKwkJICovCiAJCWZvciAoOzspIHsKIAkJCWZyZWUoZGVzY3JpcHRpb24pOwog CQkJaWYgKChkZXNjcmlwdGlvbiA9IG1hbGxvYyhkZXNjcmxlbikpICE9IE5VTEwpIHsKLSNpZmRl ZiBfX0ZyZWVCU0RfXwogCQkJCWlmcmRlc2MuaWZyX2J1ZmZlci5idWZmZXIgPSBkZXNjcmlwdGlv bjsKIAkJCQlpZnJkZXNjLmlmcl9idWZmZXIubGVuZ3RoID0gZGVzY3JsZW47Ci0jZWxzZSAvKiBf X0ZyZWVCU0RfXyAqLwotCQkJCWlmcmRlc2MuaWZyX2RhdGEgPSAoY2FkZHJfdClkZXNjcmlwdGlv bjsKLSNlbmRpZiAvKiBfX0ZyZWVCU0RfXyAqLwotCQkJCWlmIChpb2N0bChzLCBTSU9DR0lGREVT Q1IsICZpZnJkZXNjKSA9PSAwKQotCQkJCQlicmVhazsKLSNpZmRlZiBfX0ZyZWVCU0RfXwotCQkJ CWVsc2UgaWYgKGVycm5vID09IEVOQU1FVE9PTE9ORykKLQkJCQkJZGVzY3JsZW4gPSBpZnJkZXNj Lmlmcl9idWZmZXIubGVuZ3RoOwotI2VuZGlmIC8qIF9fRnJlZUJTRF9fICovCi0JCQkJZWxzZQor CQkJCWlmIChpb2N0bChzLCBTSU9DR0lGREVTQ1IsICZpZnJkZXNjKSA9PSAwKSB7CisJCQkJCWlm IChpZnJkZXNjLmlmcl9idWZmZXIuYnVmZmVyID09CisJCQkJCSAgICBkZXNjcmlwdGlvbikKKwkJ CQkJCWJyZWFrOworCQkJCQllbHNlCisJCQkJCQlkZXNjcmxlbiA9IGlmcmRlc2MuaWZyX2J1ZmZl ci5sZW5ndGg7CisJCQkJfSBlbHNlIHsKKwkJCQkJLyoKKwkJCQkJICogRmFpbGVkIHRvIGdldCBp bnRlcmZhY2UgZGVzY3JpcHRpb24uCisJCQkJCSAqLworCQkJCQlmcmVlKGRlc2NyaXB0aW9uKTsK KwkJCQkJZGVzY3JpcHRpb24gPSBOVUxMOwogCQkJCQlicmVhazsKKwkJCQl9CiAJCQl9IGVsc2UK IAkJCQlicmVhazsKIAkJfQorI2Vsc2UgLyogX19GcmVlQlNEX18gKi8KKwkJLyoKKwkJICogVGhl IG9ubHkgb3RoZXIgT1MgdGhhdCBjdXJyZW50bHkgc3VwcG9ydHMKKwkJICogU0lPQ0dJRkRFU0NS IGlzIE9wZW5CU0QsIGFuZCBpdCBoYXMgbm8gd2F5CisJCSAqIHRvIGdldCB0aGUgZGVzY3JpcHRp b24gbGVuZ3RoIC0gaXQncyBjbGFtcGVkCisJCSAqIHRvIGEgbWF4aW11bSBvZiBJRkRFU0NSU0la RS4KKwkJICovCisJCWlmICgoZGVzY3JpcHRpb24gPSBtYWxsb2MoZGVzY3JsZW4pKSAhPSBOVUxM KSB7CisJCQlpZnJkZXNjLmlmcl9kYXRhID0gKGNhZGRyX3QpZGVzY3JpcHRpb247CisJCQlpZiAo aW9jdGwocywgU0lPQ0dJRkRFU0NSLCAmaWZyZGVzYykgIT0gMCkgeworCQkJCS8qCisJCQkJICog RmFpbGVkIHRvIGdldCBpbnRlcmZhY2UgZGVzY3JpcHRpb24uCisJCQkJICovCisJCQkJZnJlZShk ZXNjcmlwdGlvbik7CisJCQkJZGVzY3JpcHRpb24gPSBOVUxMOworCQkJfQorCQl9IGVsc2UKKwkJ CWJyZWFrOworI2VuZGlmIC8qIF9fRnJlZUJTRF9fICovCiAJCWNsb3NlKHMpOwogCQlpZiAoZGVz Y3JpcHRpb24gIT0gTlVMTCAmJiBzdHJsZW4oZGVzY3JpcHRpb24pID09IDApIHsKIAkJCWZyZWUo ZGVzY3JpcHRpb24pOwo= --0016e649c770b3a94d049d87d503--