Skip to content

Commit

Permalink
Do not use std::random_device
Browse files Browse the repository at this point in the history
Directly read from urandom instead of using std::random_device.
libc++ will use iostream under-the-hood, which brings significant
binary size increase that is not welcomed, especially in magiskinit.
  • Loading branch information
topjohnwu committed Jul 15, 2019
1 parent 41045b6 commit 52fd508
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 9 deletions.
2 changes: 1 addition & 1 deletion native/jni/init/rootdir.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ static void patch_init_rc(FILE *rc) {
fprintf(rc, "%s", line.data());
return true;
});
char pfd_svc[32], ls_svc[32], bc_svc[32];
char pfd_svc[16], ls_svc[16], bc_svc[16];
gen_rand_str(pfd_svc, sizeof(pfd_svc));
gen_rand_str(ls_svc, sizeof(ls_svc));
gen_rand_str(bc_svc, sizeof(bc_svc));
Expand Down
16 changes: 8 additions & 8 deletions native/jni/utils/misc.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -53,16 +53,16 @@ int fork_no_zombie() {
constexpr char ALPHANUM[] = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789";
static bool seeded = false;
static std::mt19937 gen;
static std::uniform_int_distribution<int> dist(0, sizeof(ALPHANUM) - 1);
static std::uniform_int_distribution<int> dist(0, sizeof(ALPHANUM) - 2);
void gen_rand_str(char *buf, int len, bool varlen) {
if (!seeded) {
if (access("/dev/urandom", F_OK) == 0) {
std::random_device rdev;
gen.seed(rdev());
} else {
// In magiskinit
gen.seed(time(nullptr));
}
if (access("/dev/urandom", F_OK) != 0)
mknod("/dev/urandom", 0600 | S_IFCHR, makedev(1, 9));
int fd = xopen("/dev/urandom", O_RDONLY | O_CLOEXEC);
unsigned seed;
xxread(fd, &seed, sizeof(seed));
gen.seed(seed);
close(fd);
seeded = true;
}
if (varlen) {
Expand Down

0 comments on commit 52fd508

Please sign in to comment.