Skip to content

use config_options.default_pass_length for ns_sendpassmail.c#57

Open
jesopo wants to merge 1 commit into
masterfrom
jess/ns_sendpassmail-passlen
Open

use config_options.default_pass_length for ns_sendpassmail.c#57
jesopo wants to merge 1 commit into
masterfrom
jess/ns_sendpassmail-passlen

Conversation

@jesopo

@jesopo jesopo commented Jan 13, 2022

Copy link
Copy Markdown
Member

No description provided.

@aaronmdjones

aaronmdjones commented Jan 14, 2022

Copy link
Copy Markdown
Member

This option does not exist in version 7.2 of services; it needs some version detection logic.

diff --git a/src/ns_sendpassmail.c b/src/ns_sendpassmail.c
index d9e51d1..873448c 100644
--- a/src/ns_sendpassmail.c
+++ b/src/ns_sendpassmail.c
@@ -76,7 +76,11 @@ sendpassmail_foreach_cb(myentity_t *mt, void *privdata)
 		return 0;
 	}
 
+#if (CURRENT_ABI_REVISION < 730000)
 	key = random_string(12);
+#else
+	key = random_string(config_options.default_pass_length);
+#endif
 
 	const char *const hash = crypt_password(key);

(Edited to actually use random_string() >_>)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants