Unset / remove property src-address-list … setting it to empty string is not the same as not setting it at all.
Setting in-interface=all-wireless is problematic as well, for two reasons:
using automatic interface lists has a few quirks and it’s better to use interface names explicitly
it only works if wireless interfaces are not made bridge ports. If wireless interfaces are in fact bridge pirts, then firewall will see bridge interface as in-interface
Setting protocol=tcp means tgat this rule will not trigger on any other protocols, i.e. all remote hosts will be able to connect to router using e.g. UDP.