Merge pull request #1160 from vojtechtrefny/master_s390-fix-string-comparison

s390: Fix string comparison in bd_s390_zfcp_scsi_offline

Author: vojtechtrefny

src/plugins/s390.c

@@ -812,6 +812,7 @@ gboolean bd_s390_zfcp_scsi_offline (const gchar *devno, const gchar *wwpn, const
             g_propagate_error (error, l_error);
             return FALSE;
         }
+        g_strchomp (fcphbasysfs);
         fclose (fd);
         g_free (hba_path);
 
@@ -843,6 +844,7 @@ gboolean bd_s390_zfcp_scsi_offline (const gchar *devno, const gchar *wwpn, const
             g_propagate_error (error, l_error);
             return FALSE;
         }
+        g_strchomp (fcpwwpnsysfs);
         fclose (fd);
         g_free (wwpn_path);
 
@@ -876,14 +878,15 @@ gboolean bd_s390_zfcp_scsi_offline (const gchar *devno, const gchar *wwpn, const
             g_propagate_error (error, l_error);
             return FALSE;
         }
+        g_strchomp (fcplunsysfs);
         fclose (fd);
         g_free (lun_path);
         g_free (fcpsysfs);
 
         /* make sure read values align with expected values */
         scsidel = g_strdup_printf ("%s/%s/delete", scsidevsysfs, scsidev);
         scsidel = g_strchomp (scsidel);
-        if ((fcphbasysfs == devno) && (fcpwwpnsysfs == wwpn) && (fcplunsysfs == lun)) {
+        if (g_strcmp0 (fcphbasysfs, devno) == 0 && g_strcmp0 (fcpwwpnsysfs, wwpn) == 0 && g_strcmp0 (fcplunsysfs, lun) == 0) {
             fd = fopen (scsidel, "w");
             if (!fd) {
                 g_set_error (&l_error, BD_S390_ERROR, BD_S390_ERROR_DEVICE,