Hi,
On Tuesday 26 August 2008, Bastian Friedrich wrote:
>
> On Monday 25 August 2008, Kern Sibbald wrote:
> > This sounds like a bug to me and probably should be reported in the bugs
> > database. That said, it would be good to try to isolate what is going on
> > here as much as possible because we currently don't have much spare time,
> > which means that if you can pinpoint the problem we can probably fix it
> > quickly, but if we need reproduce the problem and research what is going
> > wrong, it will need to wait a bit.
>
[...]
>
> As I said, I was not able to track the problem down in a sensible way; for
> files not overwritten, probably there should be some "if (!extract) break;"
> lines in some (all?) of the stream examination chunks
this basically is the solution for the problem, in fact. I am attaching a
patch that fixes most of the problems described in my original posting (and
in the bug ticket).
I still have a problem, though:
When creating directories, the internal "extract" flag always is false,
independently of whether the directory was in fact created, or whether it did
already exist. The "makepath" function in findlib does not return information
about whether it created any paths. Due to this, we can only _always_ or
_never_ set directory ACLs during restore. I chose "always", but that means
that "replace=never" restores overwrite already existing ACLs.
I am not 100% sure about other constraints of directory ACLs; there still may
be issues.
The patch is restricted to ACLs; other stream types as restored in restore.c
probably have similar concerns, but I did not check that.
Feel free to use the patch; you have my signed FLA.
Thx & best regards
Bastian
--
Collax GmbH . Burkheimer Straße 3 . 79111 Freiburg . Germany
p: +49 (0) 761-45684-24
f: +49 (0) 761-45684-10 www.collax.com
Geschäftsführer: William K. Hite / Boris Nalbach
AG München HRB 158898 . Ust.-IdNr: DE 814464942
\ "Mr. Worf, scan that ship." "Aye Captain. 300 dpi?"
fix-acls-in-replace-never-vs.20080825svn.patch
Description: Text Data
-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/ _______________________________________________
Bacula-users mailing list
Bacula-users AT lists.sourceforge DOT net
https://lists.sourceforge.net/lists/listinfo/bacula-users
|