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

JSON_VALUE_PAYLOAD_MASK failed #714

Closed
Trigger-broom-289 opened this issue Feb 2, 2023 · 2 comments
Closed

JSON_VALUE_PAYLOAD_MASK failed #714

Trigger-broom-289 opened this issue Feb 2, 2023 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@Trigger-broom-289
Copy link

Describe the bug
indi_weatherradio crashing on connection with "JSON_VALUE_PAYLOAD_MASK" failed

To Reproduce
root@heidenrod-obs:~# indiserver -v indi_weatherradio
2023-02-02T21:41:59: startup: indiserver -v indi_weatherradio
2023-02-02T21:41:59: Driver indi_weatherradio: pid=1540 rfd=6 wfd=6 efd=7
2023-02-02T21:41:59: listening to port 7624 on fd 5
2023-02-02T21:41:59: Local server: listening on local domain at: @/tmp/indiserver
2023-02-02T21:41:59: Driver indi_weatherradio: snooping on GPS Simulator.GEOGRAPHIC_COORD
2023-02-02T21:42:07: Client 9: new arrival from 127.0.0.1:44970 - welcome!
2023-02-02T21:42:13: Driver indi_weatherradio: indi_weatherradio: ./indi-duino/gason/gason.h:32: JsonValue::JsonValue(JsonTag, void): Assertion `(uintptr_t)payload <= JSON_VALUE_PAYLOAD_MASK' failed.*
2023-02-02T21:42:13: Driver indi_weatherradio: read EOF

2023-02-02T21:42:13: Driver indi_weatherradio: restart #0
2023-02-02T21:42:13: Driver indi_weatherradio: pid=1546 rfd=7 wfd=7 efd=10
2023-02-02T21:42:13: Driver indi_weatherradio: snooping on GPS Simulator.GEOGRAPHIC_COORD
2023-02-02T21:42:14: Client 9: read EOF
2023-02-02T21:42:14: Client 9: shut down complete - bye!

Expected behavior
Working driver.

Desktop (please complete the following information):

  • OS: Ubuntu 22.04
  • Version [2.0.0]
    root@heidenrod-obs:~# indiserver -v
    2023-02-02T21:45:02: startup: indiserver -v
    Usage: indiserver [options] driver [driver ...]
    Purpose: server for local and remote INDI drivers
    INDI Library: 2.0.0
    Code 2.0.0-tgz. Protocol 1.7.
@Trigger-broom-289 Trigger-broom-289 added the bug Something isn't working label Feb 2, 2023
@Trigger-broom-289
Copy link
Author

Resolved on an Arm64 board.

In https://github.com/sterne-jaeger/indi-3rdparty/blob/master/indi-duino/gason/gason.h

replace
#define JSON_VALUE_PAYLOAD_MASK 0x00007FFFFFFFFFFFULL
#define JSON_VALUE_NAN_MASK 0x7FF8000000000000ULL
#define JSON_VALUE_TAG_MASK 0xF
#define JSON_VALUE_TAG_SHIFT 47

with
#define JSON_VALUE_PAYLOAD_MASK 0x0000FFFFFFFFFFFFULL
#define JSON_VALUE_NAN_MASK 0x7FF0000000000000ULL
#define JSON_VALUE_TAG_MASK 0xF
#define JSON_VALUE_TAG_SHIFT 48

@Trigger-broom-289
Copy link
Author

I can now connect to the indi_weatherradio driver.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants