Skip to content
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

!!!Bug: print PX4 version error!!! #1151

Closed
xiaofyao opened this issue Jan 3, 2019 · 0 comments
Closed

!!!Bug: print PX4 version error!!! #1151

xiaofyao opened this issue Jan 3, 2019 · 0 comments

Comments

@xiaofyao
Copy link

xiaofyao commented Jan 3, 2019

This is only bug and feature tracker, please use it
to report bugs or request features.


Issue details

When the mavros library printed the PX4 version, I found that the result was wrong.

I found this bug by investigating the source code:

https://github.com/mavlink/mavros/blob/master/mavros/src/plugins/sys_status.cpp#L583

Error passing parameter in this place. I think the correct way to pass parameters is as follows:
return utils::format("%016llX", b);

@vooon vooon closed this as completed in bdc8ab1 Jan 3, 2019
@vooon vooon added this to the Version 0.28 milestone Jan 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants