diff --git a/business/jxstore/act/act.go b/business/jxstore/act/act.go
index 8bfca5791..d2fa1d696 100644
--- a/business/jxstore/act/act.go
+++ b/business/jxstore/act/act.go
@@ -885,11 +885,13 @@ func DeleteActStoreSkuBind(ctx *jxcontext.Context, db *dao.DaoDB, actID int, act
 	if isNeedCancelAct {
 		act := &model.Act{}
 		act.ID = actID
-		if _, err = dao.UpdateEntityLogically(db, act,
-			map[string]interface{}{
-				model.FieldStatus: model.ActStatusCanceled,
-			}, ctx.GetUserName(), nil); err != nil {
-			return 0, err
+		if err == nil {
+			if _, err = dao.UpdateEntityLogically(db, act,
+				map[string]interface{}{
+					model.FieldStatus: model.ActStatusCanceled,
+				}, ctx.GetUserName(), nil); err != nil {
+				return 0, err
+			}
 		}
 	}
 	dao.Commit(db)