octoleo/restic
2
2
Fork 0
mirror of https://github.com/octoleo/restic.git synced 2024-12-22 10:58:55 +00:00
Code Issues Releases Wiki Activity

restorer: Make sure node names are clean

Browse Source
This commit is contained in:
Alexander Neumann 2017-11-26 15:17:12 +01:00
parent 8a171731ba
commit 27d29b9853
1 changed files with 43 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

62
internal/restic/restorer.go
View File

@ -39,19 +39,47 @@ func NewRestorer(repo Repository, id ID) (*Restorer, error) {
return r, nil return r, nil
} }
func (res *Restorer) restoreTo(ctx context.Context, dst string, dir string, treeID ID, idx *HardlinkIndex) error { // restoreTo restores a tree from the repo to a destination. target is the path in
// the file system, location within the snapshot.
func (res *Restorer) restoreTo(ctx context.Context, target, location string, treeID ID, idx *HardlinkIndex) error {
debug.Log("%v %v %v", target, location, treeID.Str())
tree, err := res.repo.LoadTree(ctx, treeID) tree, err := res.repo.LoadTree(ctx, treeID)
if err != nil { if err != nil {
return res.Error(dir, nil, err) debug.Log("error loading tree %v: %v", treeID.Str(), err)
return res.Error(location, nil, err)
} }
for _, node := range tree.Nodes { for _, node := range tree.Nodes {
selectedForRestore, childMayBeSelected := res.SelectFilter(filepath.Join(dir, node.Name),
filepath.Join(dst, dir, node.Name), node) // ensure that the node name does not contain anything that refers to a
// top-level directory.
nodeName := filepath.Base(filepath.Join(string(filepath.Separator), node.Name))
if nodeName != node.Name {
debug.Log("node %q has invalid name %q", node.Name, nodeName)
err := res.Error(location, node, errors.New("node has invalid name"))
if err != nil {
return err
}
continue
}
nodeTarget := filepath.Join(target, nodeName)
nodeLocation := filepath.Join(location, nodeName)
if target == nodeTarget || !fs.HasPathPrefix(target, nodeTarget) {
debug.Log("node %q has invalid target path %q", node.Name, nodeTarget)
err := res.Error(nodeLocation, node, errors.New("node has invalid path"))
if err != nil {
return err
}
continue
}
selectedForRestore, childMayBeSelected := res.SelectFilter(nodeLocation, nodeTarget, node)
debug.Log("SelectFilter returned %v %v", selectedForRestore, childMayBeSelected) debug.Log("SelectFilter returned %v %v", selectedForRestore, childMayBeSelected)
if selectedForRestore { if selectedForRestore {
err = res.restoreNodeTo(ctx, node, dir, dst, idx) err = res.restoreNodeTo(ctx, node, nodeTarget, nodeLocation, idx)
if err != nil { if err != nil {
return err return err
} }
@ -62,10 +90,9 @@ func (res *Restorer) restoreTo(ctx context.Context, dst string, dir string, tree
return errors.Errorf("Dir without subtree in tree %v", treeID.Str()) return errors.Errorf("Dir without subtree in tree %v", treeID.Str())
} }
subp := filepath.Join(dir, node.Name) err = res.restoreTo(ctx, nodeTarget, nodeLocation, *node.Subtree, idx)
err = res.restoreTo(ctx, dst, subp, *node.Subtree, idx)
if err != nil { if err != nil {
err = res.Error(subp, node, err) err = res.Error(nodeLocation, node, err)
if err != nil { if err != nil {
return err return err
} }
@ -74,7 +101,7 @@ func (res *Restorer) restoreTo(ctx context.Context, dst string, dir string, tree
if selectedForRestore { if selectedForRestore {
// Restore directory timestamp at the end. If we would do it earlier, restoring files within // Restore directory timestamp at the end. If we would do it earlier, restoring files within
// the directory would overwrite the timestamp of the directory they are in. // the directory would overwrite the timestamp of the directory they are in.
err = node.RestoreTimestamps(filepath.Join(dst, dir, node.Name)) err = node.RestoreTimestamps(nodeTarget)
if err != nil { if err != nil {
return err return err
} }
@ -85,13 +112,12 @@ func (res *Restorer) restoreTo(ctx context.Context, dst string, dir string, tree
return nil return nil
} }
func (res *Restorer) restoreNodeTo(ctx context.Context, node *Node, dir string, dst string, idx *HardlinkIndex) error { func (res *Restorer) restoreNodeTo(ctx context.Context, node *Node, target, location string, idx *HardlinkIndex) error {
debug.Log("node %v, dir %v, dst %v", node.Name, dir, dst) debug.Log("%v %v %v", node.Name, target, location)
dstPath := filepath.Join(dst, dir, node.Name)
err := node.CreateAt(ctx, dstPath, res.repo, idx) err := node.CreateAt(ctx, target, res.repo, idx)
if err != nil { if err != nil {
debug.Log("node.CreateAt(%s) error %v", dstPath, err) debug.Log("node.CreateAt(%s) error %v", target, err)
} }
// Did it fail because of ENOENT? // Did it fail because of ENOENT?
@ -99,22 +125,20 @@ func (res *Restorer) restoreNodeTo(ctx context.Context, node *Node, dir string,
debug.Log("create intermediate paths") debug.Log("create intermediate paths")
// Create parent directories and retry // Create parent directories and retry
err = fs.MkdirAll(filepath.Dir(dstPath), 0700) err = fs.MkdirAll(filepath.Dir(target), 0700)
if err == nil || os.IsExist(errors.Cause(err)) { if err == nil || os.IsExist(errors.Cause(err)) {
err = node.CreateAt(ctx, dstPath, res.repo, idx) err = node.CreateAt(ctx, target, res.repo, idx)
} }
} }
if err != nil { if err != nil {
debug.Log("error %v", err) debug.Log("error %v", err)
err = res.Error(dstPath, node, err) err = res.Error(location, node, err)
if err != nil { if err != nil {
return err return err
} }
} }
debug.Log("successfully restored %v", node.Name)
return nil return nil
} }