search

tgxn / Backup

A Backup plugin for the Bukkit Server API.
http://bukkitbackup.com
GNU General Public License v3.0
23 stars 14 forks source link

Read Only Access Generates NullPointerException #69

Closed iain-davis closed 12 years ago

iain-davis commented 12 years ago

I wasn't paying attention and set the backup filepath to a remote share that was read-only for the account that was logged in. I know--obviously I shouldn't set the path to something that can't be written to--but I imagine it would better to handle the error gracefully...

Here's the error that was generated:

18:58:51 [SEVERE] java.lang.NullPointerException
18:58:51 [SEVERE]       at net.tgxn.bukkit.backup.utils.LogUtils.exceptionLog(LogUtils.java:68)
18:58:51 [SEVERE]       at net.tgxn.bukkit.backup.threading.BackupTask.backupWorlds(BackupTask.java:202)
18:58:51 [SEVERE]       at net.tgxn.bukkit.backup.threading.BackupTask.doBackup(BackupTask.java:126)
18:58:51 [SEVERE]       at net.tgxn.bukkit.backup.threading.BackupTask.run(BackupTask.java:70)
18:58:51 [SEVERE]       at org.bukkit.craftbukkit.scheduler.CraftWorker.run(CraftWorker.java:34)
18:58:51 [SEVERE]       at java.lang.Thread.run(Unknown Source)
iain-davis commented 12 years ago

Oh! I meant to include that this was with Bukkit 1.1-R3 and Backup v1.8.6.

tgxn commented 12 years ago

Assigned milestone and priority, will post fix in ~5 minutes.

tgxn commented 12 years ago

See if that fixed it :)

(I'm very lazy at this time of the night)

tgxn commented 12 years ago

Closing as should be fixed, re-open if not.