[cvsnt] Core dumps on Solaris 10

[Tom Halliley]

In src/server.cpp, is a null check missing?  Here's the method:


    static int client_protocol_buffer_flush (void *closure)
    {
        client_protocol_data *data = (client_protocol_data*)closure;
        if(data->protocol && data->protocol->server_flush_data)
            data->protocol->server_flush_data(data->protocol);

        return 0;
    }


that is, should

    if(data->protocol && data->protocol->server_flush_data)

be

    if(data && data->protocol && data->protocol->server_flush_data)


??

Tom