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

Some Magic numbers correction in Comparator for #196 #199

Merged
merged 3 commits into from
May 22, 2017

Conversation

mnaza
Copy link
Contributor

@mnaza mnaza commented May 21, 2017

No description provided.

@mnaza mnaza self-assigned this May 21, 2017
@mnaza mnaza requested review from ignatk and storojs72 May 21, 2017 10:11
@storojs72
Copy link
Contributor

Good. I'm waiting response from @secumod about some crypto questions and then will add useful comments to the code

@storojs72 storojs72 merged commit 2efa892 into cossacklabs:master May 22, 2017
@mnaza mnaza deleted the comparator_issue_196 branch May 22, 2017 06:52
@@ -23,6 +23,9 @@ static themis_status_t secure_comparator_alice_step3(secure_comparator_t *comp_c
static themis_status_t secure_comparator_bob_step4(secure_comparator_t *comp_ctx, const void *input, size_t input_length, void *output, size_t *output_length);
static themis_status_t secure_comparator_alice_step5(secure_comparator_t *comp_ctx, const void *input, size_t input_length, void *output, size_t *output_length);

#define HASH_LENGTH 64 //for sha512
#define TEMP_BUFFER_LENGTH ((ED25519_GE_LENGTH>HASH_LENGTH)?ED25519_GE_LENGTH:HASH_LENGTH)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spaces please!

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.

3 participants