Bug Fix and Formatting

Translate null characters to blanks when not preceded by ">" to prevent premature string termination.

Signed-off-by: Elijah Swift <elijah.swift@ibm.com>

Author: ElijahSwiftIBM

pyracf/common/irrsmo00.c

@@ -55,6 +55,17 @@ static PyObject* call_irrsmo00(PyObject* self, PyObject* args, PyObject *kwargs)
         rsp
     );
 
+   int i;
+   for (i = 1; i < rsp_len; i++){
+        if (rsp[i] == 0) {
+	        if (rsp[i-1] == '>') {
+                        break; 
+                }
+	        else {
+                        rsp[i] = ' ';
+                }
+	}
+   }
    return Py_BuildValue("y", rsp);
 }
 

pyracf/data_set/data_set_admin.py

@@ -110,7 +110,9 @@ def add(
             self._build_xml_segments(data_set_request)
             return self._make_request(data_set_request)
         try:
-            profile = self.extract(data_set, volume=volume, generic=generic, profile_only=True)
+            profile = self.extract(
+                data_set, volume=volume, generic=generic, profile_only=True
+            )
             if self._get_field(profile, "base", "name") == data_set.lower():
                 raise AddOperationError(data_set, self._profile_type)
         except SecurityRequestError as exception:
@@ -135,7 +137,9 @@ def alter(
             self._build_xml_segments(data_set_request, alter=True)
             return self._make_request(data_set_request, irrsmo00_precheck=True)
         try:
-            profile = self.extract(data_set, volume=volume, generic=generic, profile_only=True)
+            profile = self.extract(
+                data_set, volume=volume, generic=generic, profile_only=True
+            )
         except SecurityRequestError:
             raise AlterOperationError(data_set, self._profile_type)
         if not self._get_field(profile, "base", "name") == data_set.lower():

pyracf/resource/resource_admin.py

@@ -234,7 +234,10 @@ def alter_resource_class(
     def extract_resource_class(self, class_name: str) -> Union[dict, bytes]:
         """Extract the attributes of a general resource class."""
         profile = self.extract(
-            resource=class_name, class_name="CDT", segments=["cdtinfo"], profile_only=True
+            resource=class_name,
+            class_name="CDT",
+            segments=["cdtinfo"],
+            profile_only=True,
         )
         return profile["cdtinfo"]
 
@@ -262,7 +265,10 @@ def alter_started_task(
     def extract_started_task(self, started_task_name: str) -> Union[dict, bytes]:
         """Extract the attributes of a started task profile."""
         profile = self.extract(
-            resource=started_task_name, class_name="STARTED", segments=["stdata"], profile_only=True
+            resource=started_task_name,
+            class_name="STARTED",
+            segments=["stdata"],
+            profile_only=True,
         )
         return profile["stdata"]
 
@@ -295,7 +301,10 @@ def extract_custom_field(
         """Extract the attributes of a custom field."""
         full_profile_name = f"{custom_field_type}.csdata.{custom_field_name}"
         profile = self.extract(
-            resource=full_profile_name, class_name="CFIELD", segments=["cfdef"], profile_only=True
+            resource=full_profile_name,
+            class_name="CFIELD",
+            segments=["cfdef"],
+            profile_only=True,
         )
         return profile["cfdef"]
 
@@ -326,7 +335,10 @@ def alter_kerberos_realm(
     def extract_kerberos_realm(self, kerberos_realm_name: str) -> Union[dict, bytes]:
         """Extract the attributes of a kerberos realm profile."""
         profile = self.extract(
-            resource=kerberos_realm_name, class_name="REALM", segments=["kerb"], profile_only=True
+            resource=kerberos_realm_name,
+            class_name="REALM",
+            segments=["kerb"],
+            profile_only=True,
         )
         return profile["kerb"]
 
@@ -362,7 +374,10 @@ def alter_signed_program(
     def extract_signed_program(self, signed_program_name: str) -> Union[dict, bytes]:
         """Extract the attributes of a signed program profile."""
         profile = self.extract(
-            resource=signed_program_name, class_name="PROGRAM", segments=["sigver"], profile_only=True
+            resource=signed_program_name,
+            class_name="PROGRAM",
+            segments=["sigver"],
+            profile_only=True,
         )
         profile["sigver"]["library"] = profile["base"].get("member")
         return profile["sigver"]
@@ -396,7 +411,10 @@ def extract_appc_session(
         """Extract the attributes of a APPC session profile."""
         full_profile_name = f"{net_id}.{local_lu}.{partner_lu}"
         profile = self.extract(
-            resource=full_profile_name, class_name="APPCLU", segments=["session"], profile_only=True
+            resource=full_profile_name,
+            class_name="APPCLU",
+            segments=["session"],
+            profile_only=True,
         )
         return profile["session"]