Message ID | af3734d26265712334e0cba48f8b5876c2f0b7e3-HTML@gerrit.openvpn.net |
---|---|
State | New |
Headers | show
Return-Path: <openvpn-devel-bounces@lists.sourceforge.net> Delivered-To: patchwork@openvpn.net Received: by 2002:a05:7000:2285:b0:60a:d70a:d3c7 with SMTP id o5csp213256mag; Wed, 12 Mar 2025 14:52:48 -0700 (PDT) X-Forwarded-Encrypted: i=2; AJvYcCXpmFRcic/hWC2vctmI4l9ayaEAUmsbngmvtvRCj/6LgHDHLjsqywR2rY1jLybFj3cvsEwxjvQeO7I=@openvpn.net X-Google-Smtp-Source: AGHT+IElF+ND7Vem5VYDMbNimgTt+yxYNsz3rXKdSNcgnlddi/XvTybbe0/Uq4sieOJwcGnpWtql X-Received: by 2002:a9d:7447:0:b0:72a:6f75:37e8 with SMTP id 46e09a7af769-72bb17d8444mr121664a34.0.1741816368543; Wed, 12 Mar 2025 14:52:48 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1741816368; cv=none; d=google.com; s=arc-20240605; b=IY+e/nNsM6NcTaIj7s8/OQztlubUtJrF74ja+GxPwZTqyMjo0Se+oXN3fsumEhPWFP xXE/BUA+0uqvTMQtD26PXZSKtM01CkUDK3tUOIgvs23VnAqOo+Cywry6Tt9MKbUDcGSI W5VtBOTDZGrPIjIEhc3mrgwbW3IVPvJbLCkqjFGGy9GOMOjvn2JVGp0pDsCSVBQ2S39f x4Byyag07NsrpHh3Y4o1wd1+8rfIM+54F6lGx16sE6mocUSXJKAYPyCemJwzOLgMzr48 8YdDIiJfZAqvRIz2CjEQkTSjOc7S1exF2h5bFCXBdNFlSlA5+4KyZZy/wKA3pedz4gEE /UCA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20240605; h=errors-to:cc:reply-to:list-subscribe:list-help:list-post :list-archive:list-unsubscribe:list-id:precedence:subject:user-agent :mime-version:message-id:references:auto-submitted:to:date:from :dkim-signature:dkim-signature:dkim-signature; bh=2k8u46/rQVhiYfoNz4mqFh5QvA59IvH+yeDFzi7i/dQ=; fh=U7wEyxtwz2o5+UdevFSA47vNeG9knhWH0KV//QhD5a0=; b=cxZgDruBjJ6eidRPLAoUtQLbtv6nKbmsGRTNTRKfjTZLt94MGpqE1Jx5zxbzTzfCqO uS4UuoY+VIHlNx4i2BbTKQBrU8B99oFGH1wbikCutubIhbbjny4/f2k5MMmozhEl9F0/ BJGu6qYB7z2ULCbNMSZpQSn6YUZUj0Ue8gCz1DYI/Rpw9j5PmsQm5Q0vchbKVrwmGfUI YXlzO3nERuv9AA6fEszifUcfgDmtfZpvYZaqRGALdYzTbuWep+0pubkfVI5bc/tOJZDJ zDyyCDR1ECXGzrVIhAvE75bWCwf8L/7DkjiqpxsnMLvE9xj2EM5pdsYcaPK7LCvKLWCL PpZA==; dara=google.com ARC-Authentication-Results: i=1; mx.google.com; dkim=neutral (body hash did not verify) header.i=@sourceforge.net header.s=x header.b=Nx5awfCN; dkim=neutral (body hash did not verify) header.i=@sf.net header.s=x header.b=UQtxtiUg; dkim=neutral (body hash did not verify) header.i=@openvpn.net header.s=google header.b=SFAc0AR7; spf=pass (google.com: domain of openvpn-devel-bounces@lists.sourceforge.net designates 216.105.38.7 as permitted sender) smtp.mailfrom=openvpn-devel-bounces@lists.sourceforge.net; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=openvpn.net; dara=fail header.i=@openvpn.net Received: from lists.sourceforge.net (lists.sourceforge.net. [216.105.38.7]) by mx.google.com with ESMTPS id 46e09a7af769-72b9cb39835si2884760a34.248.2025.03.12.14.52.48 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Wed, 12 Mar 2025 14:52:48 -0700 (PDT) Received-SPF: pass (google.com: domain of openvpn-devel-bounces@lists.sourceforge.net designates 216.105.38.7 as permitted sender) client-ip=216.105.38.7; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@sourceforge.net header.s=x header.b=Nx5awfCN; dkim=neutral (body hash did not verify) header.i=@sf.net header.s=x header.b=UQtxtiUg; dkim=neutral (body hash did not verify) header.i=@openvpn.net header.s=google header.b=SFAc0AR7; spf=pass (google.com: domain of openvpn-devel-bounces@lists.sourceforge.net designates 216.105.38.7 as permitted sender) smtp.mailfrom=openvpn-devel-bounces@lists.sourceforge.net; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=openvpn.net; dara=fail header.i=@openvpn.net Received: from [127.0.0.1] (helo=sfs-ml-2.v29.lw.sourceforge.com) by sfs-ml-2.v29.lw.sourceforge.com with esmtp (Exim 4.95) (envelope-from <openvpn-devel-bounces@lists.sourceforge.net>) id 1tsU0J-0003bt-8B; Wed, 12 Mar 2025 21:52:44 +0000 Received: from [172.30.29.66] (helo=mx.sourceforge.net) by sfs-ml-2.v29.lw.sourceforge.com with esmtps (TLS1.2) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.95) (envelope-from <gerrit@openvpn.net>) id 1tsU0H-0003bl-P1 for openvpn-devel@lists.sourceforge.net; Wed, 12 Mar 2025 21:52:42 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sourceforge.net; s=x; h=Content-Type:Content-Transfer-Encoding:MIME-Version :Message-ID:Reply-To:References:Subject:List-Unsubscribe:List-Id:Cc:To:Date: From:Sender:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:List-Help: List-Subscribe:List-Post:List-Owner:List-Archive; bh=IgM9FvxP4eULvW1Ql2Zb8pnJXXw3x5DqNAGlheSV13k=; b=Nx5awfCNhu/MO814titU2ZjRzp hEpPQTVKUVQXoujEJTuIaEK0Uj3+Bv8HTHrZOohCeDG4+TVwrq29LCbDaXz36qsJzCNI7RJ3sd2D6 E5Y1globX4OlAOjSrfk/Np/A6dAHJHO25yZLSKlcy9iichJGwMXhfcWYlGnvynyn2T1M=; DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sf.net; s=x ; h=Content-Type:Content-Transfer-Encoding:MIME-Version:Message-ID:Reply-To: References:Subject:List-Unsubscribe:List-Id:Cc:To:Date:From:Sender:Content-ID :Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To: Resent-Cc:Resent-Message-ID:In-Reply-To:List-Help:List-Subscribe:List-Post: List-Owner:List-Archive; bh=IgM9FvxP4eULvW1Ql2Zb8pnJXXw3x5DqNAGlheSV13k=; b=U QtxtiUgbVW1b7jQVL9wwA2S804RUqy5yboHX8gmaoGDVDiyJJ9DDHvpri+TL71NVf40iSHoq+19FJ PnZgHFfy21c7G8RGPvtq6oVhCukUaMlorkFlojmWAjnQIAnJ0ucX8kZ3L0h5DUDSbP24/OQcHDMKj Gqi441P91St7FFkI=; Received: from mail-wr1-f41.google.com ([209.85.221.41]) by sfi-mx-2.v28.lw.sourceforge.com with esmtps (TLS1.2:ECDHE-RSA-AES128-GCM-SHA256:128) (Exim 4.95) id 1tsU0C-0007gy-In for openvpn-devel@lists.sourceforge.net; Wed, 12 Mar 2025 21:52:42 +0000 Received: by mail-wr1-f41.google.com with SMTP id ffacd0b85a97d-39141ffa9fcso287354f8f.0 for <openvpn-devel@lists.sourceforge.net>; Wed, 12 Mar 2025 14:52:36 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=openvpn.net; s=google; t=1741816350; x=1742421150; darn=lists.sourceforge.net; h=user-agent:content-disposition:content-transfer-encoding :mime-version:message-id:reply-to:references:subject :list-unsubscribe:list-id:auto-submitted:cc:to:date:from:from:to:cc :subject:date:message-id:reply-to; bh=IgM9FvxP4eULvW1Ql2Zb8pnJXXw3x5DqNAGlheSV13k=; b=SFAc0AR7n/ADSDM2rQ/tZQhqEFdriPCCk5hg/2nj7mmN9yA9KgVqyTDS6g6e3SMAqD cgKsOm1WSuj9ojQLXKjlziglKUccMI8eSSDmYexaatKSM739eDZ7Sr83GDX9/XTLw6qK 9gvDOzAH6s/Vp8f4of5CaW/Fm/uH5rtil2GV7WC7hsDqbY3XMQaxVhPzgL3+WmLgm+dG zm7gtwaTx286Amzf9kFXcwiQYEgF1Ij/YH3Pd9qxjoGQQp9moCZj+j6kLS4v8/oGSgLJ c8fHvh/MquOYhNPl6sHuMSabUgWlmEHYWpbfv+JqwlUm3X+G7oQPOEnz16fQsaoOerGd yqPw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1741816350; x=1742421150; h=user-agent:content-disposition:content-transfer-encoding :mime-version:message-id:reply-to:references:subject :list-unsubscribe:list-id:auto-submitted:cc:to:date:from :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=IgM9FvxP4eULvW1Ql2Zb8pnJXXw3x5DqNAGlheSV13k=; b=WSKsEue0Vm1EGeoleirK9xF5sZeWE5NkwxaGyegsqEUpsCQq2t8phb+I/mzyfY8Ccc Hmi0OCssLxgwqI1KPgJiGxXEdEBkn0+j2sqGQfXpJopA4P2LcKEE3kVZRypY5cuRooIX lthbHegE5awE/qkID+dyxqc/PxbQi9hVTKm9d7Qgh6kBK/ppukbs1Ma3XX3MspFSqKj4 SnGVzL4WTFwG+Zgad/Ey6EXT4bDYRM1xmTTxQ/ESXd553ah7Ww2h5IjQGKRiG/oXuvp0 UlEfnAMPv9ScJ1uIlYnopw+kthi5wqoJMBRGp6sAciRP2mI9cbUuokJfxbH4ypka/kt0 /bgQ== X-Gm-Message-State: AOJu0Yy4WmJAek1g3qx1h3IlFJl/ugAwtrnv9z+06c374TeBuWD+8coJ GIEUQrzt2UK1a5/jjateFMVZkWTjwSRpWJRmv+xALkv5RA+2rBJTZ/mEbEPPXP1Cnh0xJa9NHo+ J X-Gm-Gg: ASbGncuAgMvyEXIRt6ulPvo+h0yInbQ1m/L8me61E4GESaNmrUPGdycWazSW0LBCmZW 1v9j2Ogb8lS64WKhsp9e3UIoaj54wyyPDOvFssIkDwtjU8HEZkYhJ6HVhps4NCTIz6edDBEwKO5 0Q1yBntbAC6IcysD/VlvSlbeNm+y+bFgxMck2kFYOTD23iWKkUZi3TI37ph4JcOCiKRpKXMxqpD 1zAOnseg1erSy0jvn+Ngikp97mUb3C3clVs/TtriliKp+rG0yOqU4yL/CSFUJfY9gE5lm+1N8OG +zqoExVkTktzH/RVBdUZOpICzcA1BK1LGVF627wdfVP1xcUxWNQPvUZAcFfgIAMNYcpXOqMRB4K 0WscGqxyiRrxMhCZcsmcmhGL4lsZK461YiLxU X-Received: by 2002:a5d:59a4:0:b0:391:20ef:6300 with SMTP id ffacd0b85a97d-3926bfe072bmr9980293f8f.37.1741816349867; Wed, 12 Mar 2025 14:52:29 -0700 (PDT) Received: from gerrit.openvpn.in (ec2-18-159-0-78.eu-central-1.compute.amazonaws.com. [18.159.0.78]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-3912c019557sm22764401f8f.50.2025.03.12.14.52.28 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 12 Mar 2025 14:52:28 -0700 (PDT) From: "ordex (Code Review)" <gerrit@openvpn.net> X-Google-Original-From: "ordex (Code Review)" <gerrit@gerrit.openvpn.in> X-Gerrit-PatchSet: 1 Date: Wed, 12 Mar 2025 21:52:28 +0000 To: plaisthos <arne-openvpn@rfc2549.org>, flichtenheld <frank@lichtenheld.com> Auto-Submitted: auto-generated X-Gerrit-MessageType: newchange X-Gerrit-Change-Id: I27f305d3ae9bf650bab409e99173688d9f88ab65 X-Gerrit-Change-Number: 907 X-Gerrit-Project: openvpn X-Gerrit-ChangeURL: <http://gerrit.openvpn.net/c/openvpn/+/907?usp=email> X-Gerrit-Commit: 7856231f247aa56075510733f5eebe31056a37ee References: <gerrit.1741816346000.I27f305d3ae9bf650bab409e99173688d9f88ab65@gerrit.openvpn.net> Message-ID: <af3734d26265712334e0cba48f8b5876c2f0b7e3-HTML@gerrit.openvpn.net> MIME-Version: 1.0 User-Agent: Gerrit/3.8.2 X-Spam-Score: -0.2 (/) X-Spam-Report: Spam detection software, running on the system "util-spamd-1.v13.lw.sourceforge.com", has NOT identified this incoming email as spam. The original message has been attached to this so you can view it or label similar future email. If you have any questions, see the administrator of that system for details. Content preview: Attention is currently required from: flichtenheld, plaisthos. Hello plaisthos, flichtenheld, I'd like you to do a code review. Please visit Content analysis details: (-0.2 points, 6.0 required) pts rule name description ---- ---------------------- -------------------------------------------------- 0.0 RCVD_IN_VALIDITY_CERTIFIED_BLOCKED RBL: ADMINISTRATOR NOTICE: The query to Validity was blocked. See https://knowledge.validity.com/hc/en-us/articles/20961730681243 for more information. [209.85.221.41 listed in sa-accredit.habeas.com] 0.0 RCVD_IN_VALIDITY_RPBL_BLOCKED RBL: ADMINISTRATOR NOTICE: The query to Validity was blocked. See https://knowledge.validity.com/hc/en-us/articles/20961730681243 for more information. [209.85.221.41 listed in bl.score.senderscore.com] -0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at https://www.dnswl.org/, no trust [209.85.221.41 listed in list.dnswl.org] 0.0 RCVD_IN_MSPIKE_H2 RBL: Average reputation (+2) [209.85.221.41 listed in wl.mailspike.net] -0.0 SPF_PASS SPF: sender matches SPF record 0.0 SPF_HELO_NONE SPF: HELO does not publish an SPF Record 0.0 WEIRD_PORT URI: Uses non-standard port number for HTTP 0.0 HTML_MESSAGE BODY: HTML included in message -0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from author's domain 0.1 DKIM_SIGNED Message has a DKIM or DK signature, not necessarily valid -0.1 DKIM_VALID_EF Message has a valid DKIM or DK signature from envelope-from domain -0.1 DKIM_VALID Message has at least one valid DKIM or DK signature 0.0 T_KAM_HTML_FONT_INVALID Test for Invalidly Named or Formatted Colors in HTML X-Headers-End: 1tsU0C-0007gy-In Subject: [Openvpn-devel] [S] Change in openvpn[master]: socket: don't transfer bind family to socket in case of ANY address X-BeenThere: openvpn-devel@lists.sourceforge.net X-Mailman-Version: 2.1.21 Precedence: list List-Id: <openvpn-devel.lists.sourceforge.net> List-Unsubscribe: <https://lists.sourceforge.net/lists/options/openvpn-devel>, <mailto:openvpn-devel-request@lists.sourceforge.net?subject=unsubscribe> List-Archive: <http://sourceforge.net/mailarchive/forum.php?forum_name=openvpn-devel> List-Post: <mailto:openvpn-devel@lists.sourceforge.net> List-Help: <mailto:openvpn-devel-request@lists.sourceforge.net?subject=help> List-Subscribe: <https://lists.sourceforge.net/lists/listinfo/openvpn-devel>, <mailto:openvpn-devel-request@lists.sourceforge.net?subject=subscribe> Reply-To: a@unstable.cc, arne-openvpn@rfc2549.org, openvpn-devel@lists.sourceforge.net, frank@lichtenheld.com Cc: openvpn-devel <openvpn-devel@lists.sourceforge.net> Content-Type: multipart/mixed; boundary="===============5426678483144358125==" Errors-To: openvpn-devel-bounces@lists.sourceforge.net X-getmail-retrieved-from-mailbox: Inbox X-GMAIL-THRID: =?utf-8?q?1826426840384485592?= X-GMAIL-MSGID: =?utf-8?q?1826426840384485592?= X-getmail-filter-classifier: gerrit message type newchange |
Series |
[Openvpn-devel,S] Change in openvpn[master]: socket: don't transfer bind family to socket in case of ANY address
|
expand
|
diff --git a/src/openvpn/socket.c b/src/openvpn/socket.c index 6b32e30..808870b 100644 --- a/src/openvpn/socket.c +++ b/src/openvpn/socket.c @@ -1724,9 +1724,17 @@ gai_strerror(status)); } - /* the resolved 'local entry' might have a different family than what - * was globally configured */ - sock->info.af = sock->info.lsa->bind_local->ai_family; + /* the resolved family makes sense only if the host is not ANY, + * otherwise getaddrinfo() may return v4 and break connections + * to v6 only remotes + */ + if (sock->local_host) + { + /* the resolved 'local entry' might have a different family than + * what was globally configured + */ + sock->info.af = sock->info.lsa->bind_local->ai_family; + } } gc_free(&gc);
Attention is currently required from: flichtenheld, plaisthos. Hello plaisthos, flichtenheld, I'd like you to do a code review. Please visit http://gerrit.openvpn.net/c/openvpn/+/907?usp=email to review the following change. Change subject: socket: don't transfer bind family to socket in case of ANY address ...................................................................... socket: don't transfer bind family to socket in case of ANY address With the introduction of multisocket, we need to transfer the AI family of the bound address to the socket, as it may differ from what was set globally. However, this operation makes sense only when getaddrinfo() for bind is performed on a non-empty hostname. An empty hostname (ANY) may return AF_INET which will break following connection attempts to v6 only remotes. Change-Id: I27f305d3ae9bf650bab409e99173688d9f88ab65 Signed-off-by: Antonio Quartulli <antonio@mandelbit.com> --- M src/openvpn/socket.c 1 file changed, 11 insertions(+), 3 deletions(-) git pull ssh://gerrit.openvpn.net:29418/openvpn refs/changes/07/907/1