From owner-freebsd-bugs@FreeBSD.ORG Sun Jun 29 11:39:14 2008 Return-Path: Delivered-To: freebsd-bugs@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id C1F571065675; Sun, 29 Jun 2008 11:39:14 +0000 (UTC) (envelope-from shino@fornext.org) Received: from juniper.fornext.org (juniper.fornext.org [61.194.3.10]) by mx1.freebsd.org (Postfix) with ESMTP id 666968FC15; Sun, 29 Jun 2008 11:39:14 +0000 (UTC) (envelope-from shino@fornext.org) Received: from [127.0.0.1] (thyme.fornext.org [192.168.2.32]) by juniper.fornext.org (Postfix) with ESMTP id A03654503A; Sun, 29 Jun 2008 20:38:59 +0900 (JST) Date: Sun, 29 Jun 2008 20:38:51 +0900 From: Shunsuke SHINOMIYA To: Hiroki Sato In-Reply-To: <20080629.185711.61984972.hrs@allbsd.org> References: <20080629150108.6783.A2D40D1E@fornext.org> <20080629.185711.61984972.hrs@allbsd.org> Message-Id: <20080629193803.A849.A2D40D1E@fornext.org> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="------_4867660B00000000A846_MULTIPART_MIXED_" Content-Transfer-Encoding: 7bit X-Mailer: Becky! ver. 2.47 [ja] Cc: freebsd-bugs@FreeBSD.org, bug-followup@FreeBSD.org, thompsa@FreeBSD.org Subject: Re[2]: kern/125003: incorrect EtherIP header format. X-BeenThere: freebsd-bugs@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 29 Jun 2008 11:39:15 -0000 --------_4867660B00000000A846_MULTIPART_MIXED_ Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit Hi, > Could you let me know if the following patch solves your symptom or > not? It can be applied to 8.x and 7.x: > > http://people.allbsd.org/~hrs/FreeBSD/gif.diff Thank you. I applied your patch to RELENG_6_3 and modified netinet6/in6_gif.c manually(because of tab expansion?). I had 2 problems. One is syntax error around `eip_ver:4,' when BYTE_ORDER is LITTELE_ENDIAN. Another one is 2 octet padding for struct etherip_header. To solve this, I specified __packed attribute for this structure. Attached patch based on yours is for RELENG_6_3. Patched implementation works well with IX2015. 192.168.2.37: FreeBSD box 192.168.2.128: IX2015 20:22:55.540804 IP 192.168.2.128 > 192.168.2.37: etherip 76 0x0000: 4500 0060 076b 0000 4061 ecdc c0a8 0280 0x0010: c0a8 0225 3000 **** **** **** **** **** 0x0020: **** 0800 4500 003c 0cc0 0000 8001 a82e 0x0030: c0a8 0281 c0a8 0201 0800 4753 0001 0608 0x0040: 6162 6364 6566 6768 696a 6b6c 6d6e 6f70 0x0050: 7172 20:22:55.541815 IP 192.168.2.37 > 192.168.2.128: etherip 76 0x0000: 4500 0060 3511 0000 1e61 e136 c0a8 0225 0x0010: c0a8 0280 3000 **** **** **** **** **** 0x0020: **** 0800 4500 003c 319b 0000 4001 c353 0x0030: c0a8 0201 c0a8 0281 0000 4f53 0001 0608 0x0040: 6162 6364 6566 6768 696a 6b6c 6d6e 6f70 0x0050: 7172 -- Shunsuke SHINOMIYA --------_4867660B00000000A846_MULTIPART_MIXED_ Content-Type: application/octet-stream; name="gif6.diff" Content-Disposition: attachment; filename="gif6.diff" Content-Transfer-Encoding: base64 LS0tIG5ldC9pZl9naWYuaC5vcmlnCTIwMDYtMDItMDEgMDA6NTY6NDYuMDAwMDAwMDAwICswOTAw CisrKyBuZXQvaWZfZ2lmLmgJMjAwOC0wNi0yOSAxOTozNjo0MC4wMDAwMDAwMDAgKzA5MDAKQEAg LTkzLDEyICs5MywxNyBAQAogI2RlZmluZQlNVEFHX0dJRl9DQUxMRUQJMAogCiBzdHJ1Y3QgZXRo ZXJpcF9oZWFkZXIgewotCXVfaW50OF90IGVpcF92ZXI7CS8qIHZlcnNpb24vcmVzZXJ2ZWQgKi8K LQl1X2ludDhfdCBlaXBfcGFkOwkvKiByZXF1aXJlZCBwYWRkaW5nIGJ5dGUgKi8KLX07Ci0jZGVm aW5lIEVUSEVSSVBfVkVSX1ZFUlNfTUFTSyAgIDB4MGYKLSNkZWZpbmUgRVRIRVJJUF9WRVJfUlNW RF9NQVNLICAgMHhmMAotI2RlZmluZSBFVEhFUklQX1ZFUlNJT04gICAgICAgICAweDAzCisjaWYg QllURV9PUkRFUiA9PSBMSVRUTEVfRU5ESUFOCisJdV9pbnQJZWlwX3Jlc3ZsOjQsCS8qIHJlc2Vy dmVkICAqLworCQllaXBfdmVyOjQ7CS8qIHZlcnNpb24gKi8KKyNlbmRpZgorI2lmIEJZVEVfT1JE RVIgPT0gQklHX0VORElBTgorCXVfaW50CWVpcF92ZXI6NCwJLyogdmVyc2lvbiAqLworCQllaXBf cmVzdmw6NDsJLyogcmVzZXJ2ZWQgICovCisjZW5kaWYKKwl1X2ludDhfdCBlaXBfcmVzdmg7CS8q IHJlc2VydmVkICAqLworfSBfX3BhY2tlZDsKKyNkZWZpbmUgRVRIRVJJUF9WRVJTSU9OICAgICAg ICAgMHgzCiAKIC8qIFByb3RvdHlwZXMgKi8KIHZvaWQgZ2lmX2lucHV0KHN0cnVjdCBtYnVmICos IGludCwgc3RydWN0IGlmbmV0ICopOwotLS0gbmV0aW5ldC9pbl9naWYuYy5vcmlnCTIwMDYtMDIt MDEgMDA6NTY6NDYuMDAwMDAwMDAwICswOTAwCisrKyBuZXRpbmV0L2luX2dpZi5jCTIwMDgtMDYt MjkgMTk6MjI6NDguMDAwMDAwMDAwICswOTAwCkBAIC0xNDcsOCArMTQ3LDkgQEAKICNlbmRpZiAv KiBJTkVUNiAqLwogCWNhc2UgQUZfTElOSzoKICAJCXByb3RvID0gSVBQUk9UT19FVEhFUklQOwot IAkJZWlwaGRyLmVpcF92ZXIgPSBFVEhFUklQX1ZFUlNJT04gJiBFVEhFUklQX1ZFUl9WRVJTX01B U0s7Ci0gCQllaXBoZHIuZWlwX3BhZCA9IDA7CisgCQllaXBoZHIuZWlwX3ZlciA9IEVUSEVSSVBf VkVSU0lPTjsKKyAJCWVpcGhkci5laXBfcmVzdmwgPSAwOworIAkJZWlwaGRyLmVpcF9yZXN2aCA9 IDA7CiAgCQkvKiBwcmVwZW5kIEV0aGVybmV0LWluLUlQIGhlYWRlciAqLwogIAkJTV9QUkVQRU5E KG0sIHNpemVvZihzdHJ1Y3QgZXRoZXJpcF9oZWFkZXIpLCBNX0RPTlRXQUlUKTsKICAJCWlmICht ICYmIG0tPm1fbGVuIDwgc2l6ZW9mKHN0cnVjdCBldGhlcmlwX2hlYWRlcikpCi0tLSBuZXRpbmV0 Ni9pbjZfZ2lmLmMub3JpZwkyMDA2LTAyLTAxIDAwOjU2OjQ3LjAwMDAwMDAwMCArMDkwMAorKysg bmV0aW5ldDYvaW42X2dpZi5jCTIwMDgtMDYtMjkgMTk6MjQ6NDkuMDAwMDAwMDAwICswOTAwCkBA IC0xNDAsOCArMTQwLDkgQEAKICNlbmRpZgogCWNhc2UgQUZfTElOSzoKICAJCXByb3RvID0gSVBQ Uk9UT19FVEhFUklQOwotIAkJZWlwaGRyLmVpcF92ZXIgPSBFVEhFUklQX1ZFUlNJT04gJiBFVEhF UklQX1ZFUl9WRVJTX01BU0s7Ci0gCQllaXBoZHIuZWlwX3BhZCA9IDA7CisgCQllaXBoZHIuZWlw X3ZlciA9IEVUSEVSSVBfVkVSU0lPTjsKKyAJCWVpcGhkci5laXBfcmVzdmwgPSAwOworIAkJZWlw aGRyLmVpcF9yZXN2aCA9IDA7CiAgCQkvKiBwcmVwZW5kIEV0aGVybmV0LWluLUlQIGhlYWRlciAq LwogIAkJTV9QUkVQRU5EKG0sIHNpemVvZihzdHJ1Y3QgZXRoZXJpcF9oZWFkZXIpLCBNX0RPTlRX QUlUKTsKICAJCWlmIChtICYmIG0tPm1fbGVuIDwgc2l6ZW9mKHN0cnVjdCBldGhlcmlwX2hlYWRl cikpCg== --------_4867660B00000000A846_MULTIPART_MIXED_--