Skip to content

Fixed two coverity reported issues - #219

Closed
karuturi wants to merge 1 commit into
apache:masterfrom
karuturi:upload-coverity
Closed

Fixed two coverity reported issues#219
karuturi wants to merge 1 commit into
apache:masterfrom
karuturi:upload-coverity

Conversation

@karuturi

Copy link
Copy Markdown
Member

Dereference after null check
Dm: Dubious method used

Dereference after null check
Dm: Dubious method used
@asfbot

Copy link
Copy Markdown

cloudstack-pull-requests #80 SUCCESS
This pull request looks good

@DaanHoogland

Copy link
Copy Markdown
Contributor

LGTM. If you are sure the bytes are always UTF8 and not platform defined, this is fine.

@DaanHoogland

Copy link
Copy Markdown
Contributor

travis has errors in one of the jobs:( not related to the checkin ASAICT

@karuturi

Copy link
Copy Markdown
MemberAuthor

In the current usage of it, they are utf8. Also, it uses UTF-8 for the key.

I will go ahead and merge. Thanks for the review Daan.

@asfgitasfgit closed this in bd71fcbMay 1, 2015
@karuturi
karuturi deleted the upload-coverity branch May 1, 2015 10:39
yadvr pushed a commit that referenced this pull request Jan 20, 2021
Fixes#180
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
shwstppr pushed a commit to shapeblue/cloudstack that referenced this pull request Mar 13, 2023
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@karuturi@asfbot@DaanHoogland