Re: [Tipc-discussion] RE: Another tipc observation

[Jon M. <jon...@er...>]

Hi again.
I just checked it in myself, along with some other corrections.
I also added a corresonding file release, tipc-1.3.09

/jon

Mark Haverkamp wrote:


On Sat, 2004-04-10 at 11:06, Jon Maloy (QB/EMC) wrote:

  

Certainly not. I am surprised that it works at all,

given that the management code is not tested at all

yet. Feel free to correct it.



Thanks /Jon



    

OK, here is a fix that works.  I made the change in the node_get_nodes

function.  It is only called by the manager.  If you would rather, I

could fix it in mng_cmd_event but that would require casting the data

buffer to a tipc_node_info and doing the htonl there.



Mark.



--- /home/markh/views/tipc/cvs/source/unstable/net/tipc/node.c
2004-03-30 07:11:39.000000000 -0800

+++ node.c	2004-04-14 09:54:30.000000000 -0700

@@ -613,8 +613,8 @@

 	for(n = nodes;n;n = n->next) {

 		if (!in_scope(scope,n->addr))

 			continue;

-                list[cnt].addr = n->addr;

-                list[cnt].up = node_is_up(n);

+                list[cnt].addr = htonl(n->addr);

+                list[cnt].up = htonl(node_is_up(n));

                 cnt++;

 	};

 	return (cnt * sizeof(struct tipc_node_info));