headers: Add macro to print error code as string

[johnandersen777]

• Adds a PRINT_SGX_STATUS macro to sgx_error.h which prints
  the sgx_status_t as a string.

[johnandersen777]

Generated with:

(
echo -e '#define PRINT_SGX_STATUS(FILE_STRUCT, STATUS_T) \\\n    switch (STATUS_T) {\\';
for error in $(grep -E '=.*SGX_MK_ERROR' /opt/intel/sgxsdk/include/sgx_error.h | awk '{print $1}' | sed 's/=//g'); do
  printf '    case %s:\\\n        fprintf(FILE_STRUCT, "Error: %s\\n");\\\n        break;\\\n' $error $error;
done;
echo -e '    }'
)

[johnandersen777]

maybe this would be better as something similar to perror called sgx_perror or something. Where it acts just as perror does but for sgx_status_t errors

[sergefdrv]

Might be even better to make it similar to strerror, for fine control of output. Think of advanced logging mechanism.

[johnandersen777]

Ya that is probably a better option, I think that's what was started in #346 perhaps that PR should be one PR introducing the strerror and another adding the logging to various places.

[johnandersen777]

Something similar is going on in the examples

https://github.com/intel/linux-sgx/blob/a85fbe9a55767681b0f799a66555b123312fb72c/SampleCode/SampleEnclave/App/App.cpp#L55-L70

[Icaro-Lima]

Hello, I was disinclined to continue working on my pull request after the comment from the Intel member himself saying that maybe it did not help much. And I still maintain my position, I think it is very important and avoids many lines of unnecessary codes that will be inserted in each enclave.

[johnandersen777]

I agree. As I posted above something similar was done in the sample enclave code. If its useful there then it is of course useful and instead of duplicating that functionality it should be a part of the SDK.

[taraxacum45e9a]

I believe that it's a really useful patch.
FYI, here's how I did the same thing.

https://pastebin.com/PT4Ds8WQ