order clone attack

[Tio Akima 75]

hi, i made a skill to send the clone generated by the player to attack a target. (order to attack a target).
I did a custom function, where I return the clone ID on the map.
And so, give him an order.
But, I do not know if it is correct to go this way, or if there is any other necessary procedure

case CLONE_ATTACK:
		{
				int c = 0;
				int clone_id = 0;
				int maxcount = 1;
				
				map->foreachinmap(skill->check_condition_clone, sd->bl.m, BL_MOB, sd->bl.id, &c , &clone_id );	// clone search on the map
				
				//ShowWarning("clone_id: %d\n", clone_id);	//debug
		
					if( c < maxcount ) {	//if no clones are found
						clif->skill_fail(sd, skill_id, USESKILL_FAIL_CONDITION, 0, 0);
						map->freeblock_unlock();
						return 0;
					}
				struct block_list* mbl  = map->id2bl(clone_id);		//takes the clone id and converts it to a bl
				struct unit_data *ud = unit->bl2ud(mbl);			//take the bl and convert it to a unit
				
				//nullpo_ret(mbl);
				//nullpo_ret(ud);	
				
				if(ud){
					unit->set_target(ud, bl->id);					//set a target
					unit->attack(mbl, bl->id, ud->state.attack_continue); // Execute normal attack
					
				}
		}

suggestions?

 

@EDIT: Solved

Edited May 20, 2021 by Tio Akima