Do not use shell when not needed + Lint whitespaces

Author: krhubert

tasks/main.yml

@@ -72,7 +72,7 @@
     when: ansible_distribution == 'Debian' or ansible_distribution == 'Ubuntu'
 
   - name: check if ssh_password module is already installed
-    shell: "semodule -l| grep ssh_password"
+    shell: "semodule -l | grep ssh_password"
     register: ssh_password_module
     failed_when: false
     changed_when: false
@@ -100,7 +100,7 @@
 
   # The following tasks only get executed when selinux is in state enforcing, UsePam is "yes" and the ssh_password module is installed.
   - name: remove selinux-policy when Pam is used, because Allowing sshd to read the shadow file directly is considered a potential security risk (http://danwalsh.livejournal.com/12333.html)
-    shell: semodule -r ssh_password
+    command: semodule -r ssh_password
     when: ssh_use_pam and ssh_password_module.stdout.find('ssh_password') == 0
 
   when: sestatus.rc == 0