diff --git a/federatedscope/attack/worker_as_attacker/server_attacker.py b/federatedscope/attack/worker_as_attacker/server_attacker.py
index a54151c92..97c1f7e24 100644
--- a/federatedscope/attack/worker_as_attacker/server_attacker.py
+++ b/federatedscope/attack/worker_as_attacker/server_attacker.py
@@ -123,7 +123,7 @@ def callback_funcs_model_para(self, message: Message):
                         data=self.reconstruct_data[message.state][
                             message.sender][0],
                         sav_pth=self._cfg.outdir,
-                        name='image_state:{}_client:{}.png'.format(
+                        name='image_state_{}_client_{}.png'.format(
                             message.state, message.sender))
 
         self.check_and_move_on()