In the module generator.c, there is a static struct sum_struct null_sum that is not initialized by any way that I can determine in the routine write_sum_head.

Shouldn't it more correctly be:

    const struct sum_struct null_num = {0, 0, 0, 0, 0, NULL};

This would also allow the declaration for the write_sum_head routine to be as below, since the data that sum is pointing to is never modified.

    void write_sum_head(int f, const struct sum_struct * sum)

-John
[EMAIL PROTECTED]
Personal Opinion Only

--
To unsubscribe or change options: http://lists.samba.org/mailman/listinfo/rsync
Before posting, read: http://www.catb.org/~esr/faqs/smart-questions.html

Reply via email to