public inbox for b.a.t.m.a.n@lists.open-mesh.org
 help / color / mirror / Atom feed
* [B.A.T.M.A.N.] [PATCH] alfred: Keep order of interfaces from commandline
@ 2016-10-13  9:03 Sven Eckelmann
  2016-10-14 13:33 ` Simon Wunderlich
  0 siblings, 1 reply; 2+ messages in thread
From: Sven Eckelmann @ 2016-10-13  9:03 UTC (permalink / raw)
  To: b.a.t.m.a.n; +Cc: Sven Eckelmann

The first (valid) interface mac address will be used as source mac for an
dataset. But alfred added the commandline parameters for a multi-interface
master in reverse order.

Keep the list in the same order as the commandline to fulfill the
user expectations when checking the source entries of datasets.

Signed-off-by: Sven Eckelmann <sven.eckelmann@open-mesh.com>
---
 netsock.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/netsock.c b/netsock.c
index d72541e..5d37347 100644
--- a/netsock.c
+++ b/netsock.c
@@ -165,7 +165,7 @@ int netsock_set_interfaces(struct globals *globals, char *interfaces)
 			return -ENOMEM;
 		}
 
-		list_add(&interface->list, &globals->interfaces);
+		list_add_tail(&interface->list, &globals->interfaces);
 	}
 
 	return 0;

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [B.A.T.M.A.N.] [PATCH] alfred: Keep order of interfaces from commandline
  2016-10-13  9:03 [B.A.T.M.A.N.] [PATCH] alfred: Keep order of interfaces from commandline Sven Eckelmann
@ 2016-10-14 13:33 ` Simon Wunderlich
  0 siblings, 0 replies; 2+ messages in thread
From: Simon Wunderlich @ 2016-10-14 13:33 UTC (permalink / raw)
  To: b.a.t.m.a.n; +Cc: Sven Eckelmann

[-- Attachment #1: Type: text/plain, Size: 500 bytes --]

On Thursday, October 13, 2016 11:03:10 AM CEST Sven Eckelmann wrote:
> The first (valid) interface mac address will be used as source mac for an
> dataset. But alfred added the commandline parameters for a multi-interface
> master in reverse order.
> 
> Keep the list in the same order as the commandline to fulfill the
> user expectations when checking the source entries of datasets.
> 
> Signed-off-by: Sven Eckelmann <sven.eckelmann@open-mesh.com>

Applied in revision f21f747

Thanks!
     Simon

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-10-14 13:33 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-10-13  9:03 [B.A.T.M.A.N.] [PATCH] alfred: Keep order of interfaces from commandline Sven Eckelmann
2016-10-14 13:33 ` Simon Wunderlich

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox