Description of problem:
tgt-admin assumes that ACL consists only of IPs, while these can actually be just names (IQNs).

Version-Release number of selected component (if applicable):
scsi-target-utils-1.0.24-2.el6.x86_64

How reproducible:
always

Steps to Reproduce:
1. define a target in tgtd.conf with initiator-name
2. observe that with tgt-admin -s there is the name listed in ACL for the target
3. tgt-admin --update ALL -f -c /etc/tgt/targets.conf
4. command 3 exits with 22

Actual results:
exit code 22
error:
tgtadm: this access control rule does not exist
Command:
	tgtadm -C 0 --op unbind --mode target --tid 13 -I CLUSTER2
exited with code: 22.


Expected results:
ACL removed with -Q

Additional info:

example target:
<target iqn.2008-09.com.example:beaker-disk-2-d1-path1>
direct-store /dev/vg_virts/beaker-disk-2-1
initiator-name CLUSTER2
scsi_id beaker-disk-2-1
scsi_sn beaker-disk-2-1
write-cache off
</target>


The affected line in my case is line 1070 of tgt-admin:
execute("tgtadm -C $control_port --op unbind --mode target --tid $tgtadm_output_tid{$current_     target} -I $acl");

where -I should be (in my case) replaced with -Q (as it's a name and not an address). For general case probably both should be tried and exit code > 0 returned only when both fail.