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

[bot-cc1s-branch-sync]

Important

🔀 The Intel® SGX SDK codebase has moved

Thank you for your contribution!
The code this pull request modifies now resides in a new repository,¹ so, unfortunately, it can no longer be merged here as-is. If your change is still relevant, please re-open it against the SDK repository, where this code is now maintained.

If you have any questions or need assistance migrating, please ask here. As we complete the migration, older pull requests may eventually be closed. No work is lost, though, and you are welcome to re-open your change in the new repository at any time.

Footnotes

1. The affected sources were migrated from confidential-computing.sgx to the dedicated confidential-computing.sgx.sdk repo. The migration covered the SDK sources (sdk/), shared headers (common/), samples (SampleCode/), and the runtime libraries (psw/urts, psw/enclave_common). ↩