Comment in totempg_mcast in totempg.c is incorrect:

    struct totempg_mcast {
            struct totempg_mcast_header header;
            unsigned char fragmented; 
            unsigned char continuation; 
            unsigned short msg_count;
            /*
             * short msg_len[msg_count];
             */ 

To read `msg_len' area, unsigned short is used.
Wrong comment is no impact on program execution, however,
people reading the source code may be confused.

Masatake YAMATO

diff -ruN /srv/sources/src-upstream/lcopy-SOURCES/openais-trunk/exec/totempg.c 
/tmp/openais-trunk/exec/totempg.c
--- /srv/sources/src-upstream/lcopy-SOURCES/openais-trunk/exec/totempg.c        
2007-09-09 22:49:32.000000000 +0900
+++ /tmp/openais-trunk/exec/totempg.c   2008-03-12 18:22:01.000000000 +0900
@@ -128,7 +128,7 @@
        unsigned char continuation; 
        unsigned short msg_count;
        /*
-        * short msg_len[msg_count];
+        * unsigned short msg_len[msg_count];
         */ 
        /* 
         * data for messages
_______________________________________________
Openais mailing list
[email protected]
https://lists.linux-foundation.org/mailman/listinfo/openais

Reply via email to