Hi there,


MultiSync 0.80 / backup_plugin.c 1.8

the function backup_load_entries is called when a user hits the
'Options' button to configure the backup plugin, but the function
uses conn->backupdir which hasn't been configured yet - that's
why the user hit the 'Options' button!

Please find patch attached to make backup_load_entries return if
the backupdir hasn't been set.

This was causing the backup plugin to core dump on a fresh install and
probably hasn't been noticed by people who already had their backup
plugin definitions set.

Regards

marty
*** plugins/backup_plugin/src/backup_plugin.c.orig      Mon Aug 11 18:20:19 2003
--- plugins/backup_plugin/src/backup_plugin.c   Mon Aug 11 18:20:55 2003
***************
*** 123,128 ****
--- 123,131 ----
    FILE *f;
    char line[512];
  
+   /* If the backupdir hasn't been configured, skip this function */
+   if(conn->backupdir == NULL) return;
+ 
    filename = g_strdup_printf ("%s/%s", conn->backupdir, 
                              BACKUPENTRYFILE);
    if ((f = fopen(filename, "r"))) {

Reply via email to