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

Corrupted "time to expire" debug message #171

Closed
Malvineous opened this issue Nov 12, 2021 · 5 comments
Closed

Corrupted "time to expire" debug message #171

Malvineous opened this issue Nov 12, 2021 · 5 comments

Comments

@Malvineous
Copy link

Just FYI it looks like there might be a typo in one of the debug messages, I'm seeing random characters appear:

Fri Nov 12 18:17:21 2021 daemon.info smcroute[14871]: Cache timeout, flushing unused (*,G) routes!
Fri Nov 12 18:17:21 2021 daemon.debug smcroute[14871]: Checking (p??�???w?O@,ff05:fdaa::1) on br-lan, time to expire: last 60127 max 60 now: 60187
Fri Nov 12 18:17:21 2021 daemon.debug smcroute[14871]:   -> Nope, still active, valid 96 vs last valid 36.
Fri Nov 12 18:17:21 2021 daemon.debug smcroute[14871]: Checking (p??�???w?O@,ff08:1500::ca03) on br-lan, time to expire: last 60127 max 60 now: 60187
Fri Nov 12 18:17:21 2021 daemon.debug smcroute[14871]:   -> Nope, still active, valid 12978 vs last valid 4549.
Fri Nov 12 18:17:21 2021 daemon.debug smcroute[14871]: Checking (p??�???w?O@,ff08:1500::ca04) on br-lan, time to expire: last 60127 max 60 now: 60187
Fri Nov 12 18:17:21 2021 daemon.debug smcroute[14871]:   -> Nope, still active, valid 24635 vs last valid 8663.
@troglobit
Copy link
Owner

What version of SMCRoute are you using?

@Malvineous
Copy link
Author

Should be the latest release supplied by the OpenWRT project, but I don't have anything set up at the moment to try cross-compiling the git version:

$ smcrouted -v
SMCRoute v2.5.3

It's running on a MIPS CPU if that makes a difference, I think it's a 32-bit platform:

$ cat /proc/cpuinfo
system type		: MediaTek MT7620A ver:2 eco:6
machine			: Lenovo Y1
cpu model		: MIPS 24KEc V5.0
isa			: mips1 mips2 mips32r1 mips32r2

@troglobit
Copy link
Owner

Thanks, that'll save time when attempting to reproduce. Currently it looks like an uninitialized stack variable that should default to "*", so I'll probably release a fix this weekend.

@troglobit
Copy link
Owner

Found it, wasn't uninitialized, it was too small for IPv6 addresses, fixed now. Will be in the next release, likely v2.5.4

@Malvineous
Copy link
Author

Interesting, thanks for the quick fix!

troglobit added a commit that referenced this issue Nov 13, 2021
Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants