curio: make ensureOneCopy safer

curiosrc/ffi/sdr_funcs.go

@@ -168,7 +168,12 @@ func (sb *SealCalls) GenerateSDR(ctx context.Context, taskID harmonytask.TaskID,
 // ensureOneCopy makes sure that there is only one version of sector data.
 // Usually called after a successful operation was done successfully on sector data.
 func (sb *SealCalls) ensureOneCopy(ctx context.Context, sid abi.SectorID, pathIDs storiface.SectorPaths, fts storiface.SectorFileType) error {
+	if !pathIDs.HasAllSet(fts) {
+		return xerrors.Errorf("ensure one copy: not all paths are set")
+
+	}
 	for _, fileType := range fts.AllSet() {
+
 		pid := storiface.PathByType(pathIDs, fileType)
 
 		keepIn := []storiface.ID{storiface.ID(pid)}

storage/paths/local.go

@@ -739,7 +739,7 @@ func (st *Local) removeSector(ctx context.Context, sid abi.SectorID, typ storifa
 	}
 
 	spath := p.sectorPath(sid, typ)
-	log.Infof("remove %s", spath)
+	log.Infow("remove", "path", spath, "id", sid, "type", typ, "storage", storage)
 
 	if err := os.RemoveAll(spath); err != nil {
 		log.Errorf("removing sector (%v) from %s: %+v", sid, spath, err)