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

Fix msg client #14

Closed
wants to merge 7 commits into from
Closed

Fix msg client #14

wants to merge 7 commits into from

Conversation

fbrausse
Copy link
Contributor

@fbrausse fbrausse commented Aug 1, 2019

Hi,

this set of changes does the following:

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!

@gkdr
Copy link
Owner

gkdr commented Aug 11, 2019

Hi, and thank you for your contribution!

Regarding the message_client.c: It was just a first POC for myself for when I had nothing that actually uses this library yet and was broken for a long time since I didn't think it's worth the time to repair it (and in general this repository is probably only useful as an implementation example to others). So if it works, I'm sure it's good!

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.

@Neustradamus
Copy link

@fbrausse: What is the status of your PR?

@gkdr
Copy link
Owner

gkdr commented Feb 12, 2021

@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.

@gkdr gkdr changed the base branch from master to dev January 22, 2022 10:20
gkdr added a commit that referenced this pull request Jan 22, 2022
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>
@gkdr
Copy link
Owner

gkdr commented Jan 22, 2022

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!

@gkdr gkdr closed this Jan 22, 2022
@gkdr gkdr mentioned this pull request Jan 27, 2022
@hartwork
Copy link
Contributor

For anyone interested, this PR is included with commit 1c60f49 on branch dev by now.

@fbrausse
Copy link
Contributor Author

Many thanks!

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

Successfully merging this pull request may close these issues.

message_client.c does not compile
4 participants