Trigger for static analysis 20210323 #924
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The "common_types.h" file will no longer provide these.
Improve the error codes from this function. - Introduces a new error "OS_ERR_OUTPUT_TOO_LARGE" if the size limit was insufficient (instead of OS_SUCCESS). - Return OS_ERROR if an empty file was written - this likely indicates some fundamental issue with the VxWorks symbol table. - Return OS_ERR_NAME_TOO_LONG if one of the symbol names was too long, (instead of generic OS_ERROR). Improve unit test to check for/verify these responses.
Merge branches 'fix831-add_bsd_sockets_coverage', 'fix846-code_review_minor_cleanup', 'fix-888-symEach-errs' and 'fix-843-endian-cleanup' into sa-20210323
|
Can I close this? |
|
Closing old test branch. |
jphickey
added a commit
to jphickey/osal
that referenced
this pull request
Aug 10, 2022
Use the entire resource ID space (16 bits) and do not immediately recycle ID values until the full space is used. Implement a generic routine to find the next available ID, and modify every resource ID allocation to use this routine. Where necessary, this also corrects for differences in argument checking and duplicate checking between the various resource types. All resource allocation procedures now have consistent argument checking, and will reject duplicate name creation for any resources that are also associated with names.
jphickey
added a commit
to jphickey/osal
that referenced
this pull request
Aug 10, 2022
Due to a merge the setting of this to its final value got moved. Need to use "Pending" value, not the one in the table record.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Describe the contribution
CodeQL results not showing up on branch pushed to fork. Test PR for static analysis
Testing performed
N/A
Expected behavior changes
N/A
System(s) tested on
CI
Additional context
For static analysis
Third party code
N/A
Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC