[Gluster-devel] [PATCH BUG:3085] Make backupvolfile-server option actually work.

Jeff Darcy jdarcy at redhat.com
Mon Jun 27 15:20:59 UTC 2011


On 06/27/2011 11:16 AM, Devon Miller wrote:
> I was thinking more of the case where no backup server is
> configured. If the primary server fails, and a backup is not defined,
> then found=0 and err=0. So, the script will eventually exit with a 0
> when it really should have exited with a 1.

You're right, the code doesn't fix that (pre-existing) hole.  We should
really set err=1 before we check for the definition of a backup server.
 Thanks!




More information about the Gluster-devel mailing list