diff --git a/modules/passon/module.go b/modules/passon/module.go
index 5afcc11d0..adeef5033 100644
--- a/modules/passon/module.go
+++ b/modules/passon/module.go
@@ -83,7 +83,7 @@ func (this *Passon) HeroUpLv(session comm.IUserSession, heroid string, lv int32)
 			return heros[i].Lv > heros[j].Lv
 		} else {
 			conf_i, _ := this.ModuleTools.GetHeroConfig(heros[i].HeroID)
-			conf_j, _ := this.ModuleTools.GetHeroConfig(heros[i].HeroID)
+			conf_j, _ := this.ModuleTools.GetHeroConfig(heros[j].HeroID)
 			return conf_i.Color > conf_j.Color
 		}
 	})