mirror of
https://github.com/aptly-dev/aptly.git
synced 2026-06-04 05:10:40 +00:00
We should never ever overwrite files in package pool. #127
This commit is contained in:
@@ -128,16 +128,15 @@ func (pool *PackagePool) Import(path string, hashMD5 string) error {
|
||||
// unable to stat target location?
|
||||
return err
|
||||
}
|
||||
}
|
||||
} else {
|
||||
// target already exists
|
||||
if targetInfo.Size() != sourceInfo.Size() {
|
||||
// trying to overwrite file?
|
||||
return fmt.Errorf("unable to import into pool: file %s already exists", poolPath)
|
||||
}
|
||||
|
||||
if targetInfo.Size() != sourceInfo.Size() {
|
||||
// trying to overwrite file?
|
||||
return fmt.Errorf("unable to import into pool: file %s already exists", poolPath)
|
||||
}
|
||||
|
||||
if os.SameFile(targetInfo, sourceInfo) {
|
||||
// Same file on the filesystem? Just skip it.
|
||||
return err
|
||||
// assume that target is already there
|
||||
return nil
|
||||
}
|
||||
|
||||
// create subdirs as necessary
|
||||
|
||||
Reference in New Issue
Block a user