-
Notifications
You must be signed in to change notification settings - Fork 13
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
Fix msg client #14
Fix msg client #14
Conversation
Hi, and thank you for your contribution! Regarding the I have recently noticed some potential memory management issues myself while working on something else, so you might have uncovered some of the same things. From a first glance, you found even more, and I'll have to have a closer look. I'm going to do this once I get to the computer I use for working on these projects, which unfortunately might take a little while, so sorry in advance. |
@fbrausse: What is the status of your PR? |
@Neustradamus, please stop commenting on every open pr. i understand you want to help, and let me tell you that this is the opposite of helpful. if you don't, i will have to find out how to prevent you from commenting. this goes for all repositories. @fbrausse sorry, it's been such a long time that i forgot you made some very important contributions and did not in fact just fix the demo, like i thought i remembered from the title. be sure that i'm on it, though. |
this closes #14, as it is a manual merge of the changes, plus some minor additions like gitignore and changelog entries. commit 78cb8705befd1937c31133a2b27535f6e8a89475 Author: Richard Bayerle <gkdr@users.noreply.github.com> Date: Sat Jan 22 11:51:46 2022 +0100 add changelog entry commit 079cdb71daf801cbee857e37fb5b8b25dc1fcbcc Author: Richard Bayerle <gkdr@users.noreply.github.com> Date: Sat Jan 22 11:45:40 2022 +0100 add client dir to gitignore commit e2e6647e68aa5d7b8c8b7a28b502376a7c0ac99d Merge: 3872bcc c77b01c Author: Richard Bayerle <gkdr@users.noreply.github.com> Date: Sat Jan 22 11:41:09 2022 +0100 Merge branch 'fix-msg-client' of https://github.com/fbrausse/axc into fbrausse-fix-msg-client commit c77b01c Author: Franz Brauße <brausse@informatik.uni-trier.de> Date: Mon Jul 29 02:56:18 2019 +0200 src/message_client: simplify by FAIL macros commit 54e8c17 Author: Franz Brauße <brausse@informatik.uni-trier.de> Date: Thu Aug 1 12:09:00 2019 +0200 src/message_client: remove old commented handshake-based init code commit 5e4abed Author: Franz Brauße <brausse@informatik.uni-trier.de> Date: Sun Jul 28 21:29:57 2019 +0200 src/message_client: fix break loop on ^D + print newline on exit commit 4187e64 Author: Franz Brauße <brausse@informatik.uni-trier.de> Date: Sun Jul 28 21:22:28 2019 +0200 src/message_client: fix resuming session: retrieve alice's device_id commit 9ba4bb4 Author: Franz Brauße <brausse@informatik.uni-trier.de> Date: Sun Jul 28 20:55:42 2019 +0200 src/message_client: fix compiler warnings about casts between uint8_t * and char * commit 960429c Author: Franz Brauße <brausse@informatik.uni-trier.de> Date: Sun Jul 28 20:51:31 2019 +0200 axc: fix memory leaks commit 2cafb15 Author: Franz Brauße <brausse@informatik.uni-trier.de> Date: Sun Jul 28 20:51:06 2019 +0200 message_client: fix compile errors + fix memory leaks Co-authored-by: Franz Brauße <brausse@informatik.uni-trier.de> Co-authored-by: Richard Bayerle <gkdr@users.noreply.github.com>
i finally got to look at it, and the merging wasn't as bad as i feared 😬 it was a manual merge, and it looks like i have to close the PR manually too. thanks again for the contribution, @fbrausse! |
For anyone interested, this PR is included with commit 1c60f49 on branch |
Many thanks! |
Hi,
this set of changes does the following:
test/test_client.c
axc/test/test_client.c
Line 89 in 206bf43
src/message_client
(2cafb15) ...message_client
's code (c77b01c)message_client
on EOF ofstdin
(5e4abed)Especially regarding items 1 and 3 I am not completely certain that this is the intended use of the API
axc
provides. I am relatively new to the Signal protocol itself, however, the message client en- and decryption now works between Alice and Bob.Wrt. to the memory fixes on the library side (item 3 above), my assumption was that the library functions allocate this memory and therefore should also free it in order for user-code to be agnostic to future changes in the internals.
Please check carefully. I'm happy to reorder / squash or make other changes to this set of patches. Thank you!