diff --git a/routers/repo/attachment.go b/routers/repo/attachment.go
index cdb73c8a0..fea7a3384 100755
--- a/routers/repo/attachment.go
+++ b/routers/repo/attachment.go
@@ -195,6 +195,10 @@ func GetAttachment(ctx *context.Context) {
 		return
 	}
 	dataSet, err := attach.LinkedDataSet()
+	if err != nil {
+		ctx.ServerError("LinkedDataSet", err)
+		return
+	}
 
 	if repository == nil && dataSet != nil {
 		repository, _ = models.GetRepositoryByID(dataSet.RepoID)
@@ -220,11 +224,7 @@ func GetAttachment(ctx *context.Context) {
 			return
 		}
 	}
-	//dataSet, err := attach.LinkedDataSet()
-	if err != nil {
-		ctx.ServerError("LinkedDataSet", err)
-		return
-	}
+
 	if dataSet != nil {
 		isPermit, err := models.GetUserDataSetPermission(dataSet, ctx.User)
 		if err != nil {