projects
/
blerg.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Update changelog for 1.9
[blerg.git]
/
common
/
auth.c
diff --git
a/common/auth.c
b/common/auth.c
index
439e2ad
..
ef548ec
100644
(file)
--- a/
common/auth.c
+++ b/
common/auth.c
@@
-7,16
+7,19
@@
#include <string.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <stdio.h>
#include <stdlib.h>
+#include <unistd.h>
#include <crypto_scrypt.h>
#include "config.h"
#include <crypto_scrypt.h>
#include "config.h"
+#include "configuration.h"
+#include "database.h"
#include "auth.h"
#include "util.h"
#include "md5.h"
int auth_set_password(const char *username, const char *password) {
#include "auth.h"
#include "util.h"
#include "md5.h"
int auth_set_password(const char *username, const char *password) {
- char filename[
512
];
- u
nsigned char
pwhash[SCRYPT_OUTPUT_SIZE];
- u
nsigned char
salt[SCRYPT_SALT_SIZE];
+ char filename[
FILENAME_MAX
];
+ u
int8_t
pwhash[SCRYPT_OUTPUT_SIZE];
+ u
int8_t
salt[SCRYPT_SALT_SIZE];
int fd, n, r;
if (!valid_name(username) || !blerg_exists(username))
int fd, n, r;
if (!valid_name(username) || !blerg_exists(username))
@@
-35,26
+38,26
@@
int auth_set_password(const char *username, const char *password) {
read(fd, salt, SCRYPT_SALT_SIZE);
close(fd);
read(fd, salt, SCRYPT_SALT_SIZE);
close(fd);
- r = crypto_scrypt(password, n, salt, SCRYPT_SALT_SIZE, SCRYPT_N, SCRYPT_r, SCRYPT_p, pwhash, SCRYPT_OUTPUT_SIZE);
+ r = crypto_scrypt(
(const uint8_t *)
password, n, salt, SCRYPT_SALT_SIZE, SCRYPT_N, SCRYPT_r, SCRYPT_p, pwhash, SCRYPT_OUTPUT_SIZE);
if (r != 0) {
fprintf(stderr, "Failure in scrypt for %s\n", username);
return 0;
}
/* Write the password */
if (r != 0) {
fprintf(stderr, "Failure in scrypt for %s\n", username);
return 0;
}
/* Write the password */
- snprintf(filename,
512, "%s/%s/password", DATA_PATH
, username);
+ snprintf(filename,
FILENAME_MAX, "%s/%s/password", blergconf.data_path
, username);
fd = open(filename, O_WRONLY | O_CREAT, 0600);
write(fd, pwhash, SCRYPT_OUTPUT_SIZE);
close(fd);
/* Write the salt */
fd = open(filename, O_WRONLY | O_CREAT, 0600);
write(fd, pwhash, SCRYPT_OUTPUT_SIZE);
close(fd);
/* Write the salt */
- snprintf(filename,
512, "%s/%s/password_salt", DATA_PATH
, username);
+ snprintf(filename,
FILENAME_MAX, "%s/%s/password_salt", blergconf.data_path
, username);
fd = open(filename, O_WRONLY | O_CREAT, 0600);
write(fd, salt, SCRYPT_SALT_SIZE);
close(fd);
/* Mark this as a version 1 password */
fd = open(filename, O_WRONLY | O_CREAT, 0600);
write(fd, salt, SCRYPT_SALT_SIZE);
close(fd);
/* Mark this as a version 1 password */
- snprintf(filename,
512, "%s/%s/password_version", DATA_PATH
, username);
+ snprintf(filename,
FILENAME_MAX, "%s/%s/password_version", blergconf.data_path
, username);
fd = open(filename, O_WRONLY | O_CREAT, 0600);
write(fd, "1\n", 2);
close(fd);
fd = open(filename, O_WRONLY | O_CREAT, 0600);
write(fd, "1\n", 2);
close(fd);
@@
-63,71
+66,102
@@
int auth_set_password(const char *username, const char *password) {
}
int auth_get_password_version(const char *username) {
}
int auth_get_password_version(const char *username) {
- char filename[
512
];
+ char filename[
FILENAME_MAX
];
int fd;
char str[4];
int fd;
char str[4];
- int len;
+ int len
= 0
;
- s
printf(filename, "%s/%s/password_version", DATA_PATH
, username);
+ s
nprintf(filename, FILENAME_MAX, "%s/%s/password_version", blergconf.data_path
, username);
if (access(filename, F_OK) != 0) {
return 0;
}
fd = open(filename, O_RDONLY);
if (access(filename, F_OK) != 0) {
return 0;
}
fd = open(filename, O_RDONLY);
+ if (fd == -1)
+ return -1;
len = read(fd, str, 4);
close(fd);
len = read(fd, str, 4);
close(fd);
+
+ if (len < -1) {
+ perror("auth_get_password_version");
+ return -1;
+ }
+
str[len] = 0;
str[len] = 0;
+
/* strtol returns zero if there isn't a number */
return strtol(str, NULL, 10);
}
int auth_get_password(const char *username, char *password) {
/* strtol returns zero if there isn't a number */
return strtol(str, NULL, 10);
}
int auth_get_password(const char *username, char *password) {
- char filename[
512
];
+ char filename[
FILENAME_MAX
];
int fd;
int fd;
+ int read_size;
int len = 0;
if (!valid_name(username))
return 0;
int len = 0;
if (!valid_name(username))
return 0;
- sprintf(filename, "%s/%s/password", DATA_PATH, username);
- fd = open(filename, O_RDONLY);
- if (fd == -1)
- return 0;
switch(auth_get_password_version(username)) {
case 0:
switch(auth_get_password_version(username)) {
case 0:
-
len = read(fd, password, MD5_DIGEST_SIZE)
;
+
read_size = MD5_DIGEST_SIZE
;
break;
case 1:
break;
case 1:
-
len = read(fd, password, SCRYPT_OUTPUT_SIZE)
;
+
read_size = SCRYPT_OUTPUT_SIZE
;
break;
break;
+ default:
+ return 0;
}
}
+
+ snprintf(filename, FILENAME_MAX, "%s/%s/password", blergconf.data_path, username);
+ fd = open(filename, O_RDONLY);
+ if (fd == -1)
+ return 0;
+ len = read(fd, password, read_size);
close(fd);
close(fd);
+ if (len < 0) {
+ perror("auth_get_password");
+ return 0;
+ } else if (len < read_size) {
+ fprintf(stderr, "Short read getting password\n");
+ return 0;
+ }
+
password[len] = 0;
return 1;
}
password[len] = 0;
return 1;
}
-int auth_get_salt(const char *username, char *salt) {
- char filename[512];
- int fd, len;
+int auth_get_salt(const char *username, uint8_t *salt) {
+ char filename[FILENAME_MAX];
+ int fd;
+ int len = 0;
if (!valid_name(username))
return 0;
if (!valid_name(username))
return 0;
- s
printf(filename, "%s/%s/password_salt", DATA_PATH
, username);
+ s
nprintf(filename, FILENAME_MAX, "%s/%s/password_salt", blergconf.data_path
, username);
fd = open(filename, O_RDONLY);
if (fd == -1)
return 0;
len = read(fd, salt, SCRYPT_SALT_SIZE);
close(fd);
fd = open(filename, O_RDONLY);
if (fd == -1)
return 0;
len = read(fd, salt, SCRYPT_SALT_SIZE);
close(fd);
+ if (len < 0) {
+ perror("auth_get_salt");
+ return 0;
+ } else if (len < SCRYPT_SALT_SIZE) {
+ fprintf(stderr, "Short read getting salt\n");
+ return 0;
+ }
+
return 1;
}
int auth_check_password_v0(const char *username, const char *password) {
char epw[MD5_DIGEST_SIZE + 1];
return 1;
}
int auth_check_password_v0(const char *username, const char *password) {
char epw[MD5_DIGEST_SIZE + 1];
-
unsigned
char givenpw[MD5_DIGEST_SIZE];
+ char givenpw[MD5_DIGEST_SIZE];
struct MD5Context ctx;
if (auth_get_password(username, epw) == 0)
struct MD5Context ctx;
if (auth_get_password(username, epw) == 0)
@@
-136,7
+170,7
@@
int auth_check_password_v0(const char *username, const char *password) {
MD5Init(&ctx);
MD5Update(&ctx, username, strlen(username));
MD5Update(&ctx, password, strlen(password));
MD5Init(&ctx);
MD5Update(&ctx, username, strlen(username));
MD5Update(&ctx, password, strlen(password));
- MD5Final(givenpw, &ctx);
+ MD5Final(
(unsigned char *)
givenpw, &ctx);
if (strncmp(givenpw, epw, MD5_DIGEST_SIZE) == 0)
return 1;
if (strncmp(givenpw, epw, MD5_DIGEST_SIZE) == 0)
return 1;
@@
-146,17
+180,17
@@
int auth_check_password_v0(const char *username, const char *password) {
int auth_check_password_v1(const char *username, const char *password) {
unsigned char epw[SCRYPT_OUTPUT_SIZE];
int auth_check_password_v1(const char *username, const char *password) {
unsigned char epw[SCRYPT_OUTPUT_SIZE];
- u
nsigned char
esalt[SCRYPT_SALT_SIZE];
+ u
int8_t
esalt[SCRYPT_SALT_SIZE];
unsigned char givenpw[SCRYPT_OUTPUT_SIZE];
int r;
unsigned char givenpw[SCRYPT_OUTPUT_SIZE];
int r;
- if (auth_get_password(username, epw) == 0)
+ if (auth_get_password(username,
(char *)
epw) == 0)
return 0;
if (auth_get_salt(username, esalt) == 0)
return 0;
return 0;
if (auth_get_salt(username, esalt) == 0)
return 0;
- r = crypto_scrypt(password, strlen(password), esalt, SCRYPT_SALT_SIZE, SCRYPT_N, SCRYPT_r, SCRYPT_p, givenpw, SCRYPT_OUTPUT_SIZE);
+ r = crypto_scrypt(
(const uint8_t *)
password, strlen(password), esalt, SCRYPT_SALT_SIZE, SCRYPT_N, SCRYPT_r, SCRYPT_p, givenpw, SCRYPT_OUTPUT_SIZE);
if (r != 0) {
fprintf(stderr, "Failure in scrypt for %s\n", username);
return 0;
if (r != 0) {
fprintf(stderr, "Failure in scrypt for %s\n", username);
return 0;
@@
-184,6
+218,8
@@
int auth_check_password(const char *username, const char *password) {
case 1:
return auth_check_password_v1(username, password);
}
case 1:
return auth_check_password_v1(username, password);
}
+ fprintf(stderr, "auth_check_password fell through. Bad password version?\n");
+ return 0;
}
void hexify(char *dst, char *src, int len) {
}
void hexify(char *dst, char *src, int len) {
@@
-197,7
+233,7
@@
void hexify(char *dst, char *src, int len) {
}
char *create_random_token() {
}
char *create_random_token() {
-
unsigned
char buf[TOKEN_SIZE];
+ char buf[TOKEN_SIZE];
char *token;
int rand_fd;
char *token;
int rand_fd;
@@
-217,7
+253,7
@@
char *create_random_token() {
}
char * auth_login(const char *username, const char *password) {
}
char * auth_login(const char *username, const char *password) {
- char filename[
512
];
+ char filename[
FILENAME_MAX
];
int token_fd;
if (!auth_check_password(username, password))
int token_fd;
if (!auth_check_password(username, password))
@@
-225,7
+261,7
@@
char * auth_login(const char *username, const char *password) {
char *token = create_random_token();
char *token = create_random_token();
- s
printf(filename, "%s/%s/tokens", DATA_PATH
, username);
+ s
nprintf(filename, FILENAME_MAX, "%s/%s/tokens", blergconf.data_path
, username);
if (access(filename, F_OK) != 0) {
if (mkdir(filename, 0700) == -1) {
perror("Could not create auth token dir");
if (access(filename, F_OK) != 0) {
if (mkdir(filename, 0700) == -1) {
perror("Could not create auth token dir");
@@
-233,7
+269,7
@@
char * auth_login(const char *username, const char *password) {
}
}
}
}
- s
printf(filename, "%s/%s/tokens/%s", DATA_PATH
, username, token);
+ s
nprintf(filename, FILENAME_MAX, "%s/%s/tokens/%s", blergconf.data_path
, username, token);
token_fd = open(filename, O_WRONLY | O_CREAT, 0600);
if (token_fd == -1) {
perror("Could not open token");
token_fd = open(filename, O_WRONLY | O_CREAT, 0600);
if (token_fd == -1) {
perror("Could not open token");
@@
-245,17
+281,17
@@
char * auth_login(const char *username, const char *password) {
}
int auth_logout(const char *username, const char *token) {
}
int auth_logout(const char *username, const char *token) {
- char filename[
512
];
+ char filename[
FILENAME_MAX
];
if (!valid_name(username))
return 0;
if (!valid_name(username))
return 0;
- s
printf(filename, "%s/%s/tokens", DATA_PATH
, username);
+ s
nprintf(filename, FILENAME_MAX, "%s/%s/tokens", blergconf.data_path
, username);
if (access(filename, F_OK) != 0) {
return 0;
}
if (access(filename, F_OK) != 0) {
return 0;
}
- s
printf(filename, "%s/%s/tokens/%s", DATA_PATH
, username, token);
+ s
nprintf(filename, FILENAME_MAX, "%s/%s/tokens/%s", blergconf.data_path
, username, token);
if (unlink(filename) == -1)
return 0;
if (unlink(filename) == -1)
return 0;
@@
-263,9
+299,9
@@
int auth_logout(const char *username, const char *token) {
}
int auth_check_token(const char *username, const char *given_token) {
}
int auth_check_token(const char *username, const char *given_token) {
- char filename[
512
];
+ char filename[
FILENAME_MAX
];
- s
printf(filename, "%s/%s/tokens/%s", DATA_PATH
, username, given_token);
+ s
nprintf(filename, FILENAME_MAX, "%s/%s/tokens/%s", blergconf.data_path
, username, given_token);
return (access(filename, F_OK) == 0);
}
return (access(filename, F_OK) == 0);
}