Fixes static analyser issue

[valeriyvan]

New Pull Request Checklist

• I have searched for a similar pull request in the project and found none

• I have updated this branch with the latest master to avoid conflicts (via merge from master or rebase)

• I have added the required tests to prove the fix/feature I am adding

• I have updated the documentation (if necessary)

• I have run the tests and they pass

• I have run the lint and it passes (pod lib lint)

This merge request fixes / refers to the following issues: ...

Pull Request Description

...

[lolgear]

@valeriyvan

    OSStatus status = SecItemDelete((__bridge CFDictionaryRef)removeAttributes);
    if (status != errSecSuccess) {
        if (error) {
// incorrect. OSStatus is an error from different domain
            *error = [JWTErrorDescription errorWithCode:status];
        }
        return NO;
    }

[valeriyvan]

Makes sense. What would be correct way to create error here?

[valeriyvan]

Got it. It should be [NSError errorWithDomain:NSOSStatusErrorDomain code:status userInfo:nil]

[lolgear]

@valeriyvan could you check macOS tests?
I suppose that tagData == nil check is something more essential than it seems. ( It doesn't generate errors ).
In macOS it could corrupt tests.

[valeriyvan]

With or without this fix macOS tests fail. Some weird memory problems.
I guess problem is not in this fix but in somewhere else.

[wanght1029]

What did you do by clicking on correct Base64 encode? Because I need to click on the string that will be generated

[lolgear]

@valeriyvan
Sorry, It's been a long time since your PR was introduced.
If you don't mind I added this PR via combining it with another PR.
I guess it could be closed.
Could you check latest master?

Thanks!