<div dir="ltr">The while loop for -a or -r was forgetting that argv is a char**.<br><br>Also:<br><br>* sort the options in the help.<br>* remove DBASE_SIZE and the comment about keeping it up to date, since<br>  we can just use ARRAY_LEN instead (though I wonder whether anyone has<br>  enough modules to benefit from a hash table rather than just a list?).<br>* fix the comment explaining why read_line() exists.<br>* fix a typo 'aliase' for 'alias'.<br>* fix bad intentation in depmode_read_entry().<br>* simplify rm_mod() since no-one was relying on the return value being<br>  errno, the O_EXCL flag is irrelevant (but ignored by the kernel), and<br>  all callers want the same flags.<br>* simplify ins_mod() because even CentOS 7.2 has finit_module(2), and<br>  unlike insmod(1) -- which I've touched in a separate patch -- modprobe<br>  doesn't need to handle loading a module from stdin.<br>---<br> toys/pending/modprobe.c | 75 ++++++++++++-----------------------------<br> 1 file changed, 22 insertions(+), 53 deletions(-)<br></div>