Wireshark-dev: [Wireshark-dev] ws_base32_decode should be named *_encode and seems non standard
Hi all,
Something about the ws_base32_decode function looks weird, and I'm a little confused. Can I please have a sanity check? 2 things:
1. ws_base32_decode is actually an encode
2. it's not the standard base32 as defined in RFC4648
Let me explain.
1. It's actually an encode
The function is used only in one place in wireshark, in packet-fc00.c
Looking at the git history reveals that the function initially lived within that dissector, and was called `Base32_encode`. See commit 26e883a75c37e18f633e47dcd85e2648ee1f03ae
Also, the function is used by the dissector (line 139) to convert a byte array into a char array. That fits the definition of encode in most languages.
2. it's not the standard RFC4648
I tried the function, and it doesn't seem to have the same output as other base32 encoders.
With python 3.7.1: helloworld -> NBSWY3DPO5XXE3DE
On some encoder online: helloworld -> NBSWY3DPO5XXE3DE
With ws_base32_decode: helloworld -> ILZYGW5NX33EHWRM
It's also not using the same alphabet as python, so for the sake of the test I modified the code to have the same alphabet. In packet-fc00.c, there's a comment line indicating
`Cjdns style base32 encoding`
My guess is that this encoding was specific to the cjdns/fc00 protocol.
Can anyone validate, or invalidate, this reasoning?
