Fixed hipDeviceGetAttribute

1. Added negative test for hipDeviceGetAttribute
2. Fixed hipDeviceGetAttribute if int ptr input is null

Change-Id: I0e31f50fa407701fddf96e4eb64a87a371ff5d95


[ROCm/hip commit: 5b45c97a30]

projects/hip/src/hip_device.cpp

@@ -155,6 +155,8 @@ hipError_t hipDeviceGetAttribute(int* pi, hipDeviceAttribute_t attr, int device)
 
     hipError_t e = hipSuccess;
 
+    if(pi != nullptr) {
+
     auto * hipDevice = ihipGetDevice(device);
     hipDeviceProp_t *prop = &hipDevice->_props;
     if (hipDevice) {
@@ -215,6 +217,9 @@ hipError_t hipDeviceGetAttribute(int* pi, hipDeviceAttribute_t attr, int device)
     } else {
         e = hipErrorInvalidDevice;
     }
+    }else{
+        e = hipErrorInvalidDevice;
+    }
     return ihipLogStatus(e);
 }
 

projects/hip/tests/src/Functional/Negative/Device/hipDeviceGetAttribute.cpp

@@ -0,0 +1,21 @@
+#include<hip/hip_runtime.h>
+#include<hip/hip_runtime_api.h>
+#include"hipDeviceUtil.h"
+
+int main()
+{
+    int pi;
+    int attr = 0;
+//    hipDeviceAttribute_t attr = hipDeviceAttributeMaxThreadsPerBlock;
+    HIP_CHECK(hipDeviceGetAttribute(nullptr, hipDeviceAttribute_t(attr), 0), hipDeviceGetAttribute);
+    HIP_CHECK(hipDeviceGetAttribute(&pi, hipDeviceAttribute_t(attr), 0), hipDeviceGetAttribute);
+    attr = -1;
+    HIP_CHECK(hipDeviceGetAttribute(nullptr, hipDeviceAttribute_t(attr), 0), hipDeviceGetAttribute);
+    HIP_CHECK(hipDeviceGetAttribute(&pi, hipDeviceAttribute_t(attr), 0), hipDeviceGetAttribute);
+    attr = 0;
+    HIP_CHECK(hipDeviceGetAttribute(nullptr, hipDeviceAttribute_t(attr), -1), hipDeviceGetAttribute);
+    HIP_CHECK(hipDeviceGetAttribute(&pi, hipDeviceAttribute_t(attr), -1), hipDeviceGetAttribute);
+    attr = -1;
+    HIP_CHECK(hipDeviceGetAttribute(nullptr, hipDeviceAttribute_t(attr), -1), hipDeviceGetAttribute);
+    HIP_CHECK(hipDeviceGetAttribute(&pi, hipDeviceAttribute_t(attr), -1), hipDeviceGetAttribute);
+}