-
Notifications
You must be signed in to change notification settings - Fork 301
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
remoteproc/rpmsg: Wrong argument to safe_strcpy() #625
Comments
Good catch! what i cannot understand is why the CI does not detect it... |
@jonny-svaerd-arm: Please, could you have a look to the #626 fix? |
Looks good to me @arnopo. Thanks for fixing it so quickly, much appreciated. |
Thank you for reporting the issue! t was good timing as we are starting tests for the upcoming release. |
In commit c3132d0 and 27bec14 the
strncpy
function is replaced by a newsafe_strcpy
function, however the last argument to some of the calls are made withsizeof(name)
wherename
is aconst char *
(resulting in pointer size).I'm guessing these most likely were intended to be
strlen(name)
?Thanks!
The text was updated successfully, but these errors were encountered: