[Cluster-devel] cluster/ccs/daemon cnx_mgr.c

[rohara]

CVSROOT:        /cvs/cluster
Module name:    cluster
Branch:         RHEL5
Changes by:     [EMAIL PROTECTED]       2007-12-11 20:57:24

Modified files:
        ccs/daemon     : cnx_mgr.c 

Log message:
        BZ 418741
        Fix issue with endian conversion that caused problems for mixed 
architecture
        nodes on same subnet. Need to correct swap byte ordering of 
comm_header_t
        structure before copying into buffer and when dereferencing.

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/cluster/ccs/daemon/cnx_mgr.c.diff?cvsroot=cluster&only_with_tag=RHEL5&r1=1.41.2.2&r2=1.41.2.3

--- cluster/ccs/daemon/cnx_mgr.c        2007/05/08 14:26:18     1.41.2.2
+++ cluster/ccs/daemon/cnx_mgr.c        2007/12/11 20:57:24     1.41.2.3
@@ -1365,18 +1365,19 @@
     ch->comm_flags |= COMM_BROADCAST_FROM_QUORATE;
   }
 
+  swab_header(ch);
   memcpy(buffer, ch, sizeof(comm_header_t));
+  swab_header(ch); /* Swab back to dip into ch for payload_size */
   memcpy(buffer+sizeof(comm_header_t), payload, ch->comm_payload_size);
 
   log_dbg("Sending cluster.conf (version %d)...\n", 
get_doc_version(master_doc->od_doc));
   sendlen = ch->comm_payload_size + sizeof(comm_header_t);
-  swab_header(ch);
   if(sendto(sfd, buffer, sendlen, 0,
            (struct sockaddr *)&addr, (socklen_t)len) < 0){
     log_sys_err("Sendto failed");
     error = -errno;
   }
-  
+
  fail:
   if(buffer) free(buffer);
   if(payload) free(payload);