diff options
| author | Denys Vlasenko | 2015-10-24 04:17:04 +0200 |
|---|---|---|
| committer | Denys Vlasenko | 2015-10-24 04:17:04 +0200 |
| commit | cd13974b201972ffb605e243f63f674e95b99e5c (patch) | |
| tree | cbc62c9de34aab6065d76074e4369903b98f679b /modutils/modutils.c | |
| parent | 2e9a0662bce58b0fe838f7e1e03c35c4765ff3bc (diff) | |
| download | busybox-cd13974b201972ffb605e243f63f674e95b99e5c.zip busybox-cd13974b201972ffb605e243f63f674e95b99e5c.tar.gz | |
rmmod: fix bad error message
Before:
># busybox_old rmmod gtrhfhdfghdf
rmmod: can't unload 'gtrhfhdfghdf': unknown symbol in module, or unknown parameter
After:
># busybox rmmod gtrhfhdfghdf
rmmod: can't unload module 'gtrhfhdfghdf': No such file or directory
function old new delta
modprobe_main 726 721 -5
do_modprobe 599 590 -9
rmmod_main 187 169 -18
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 0/3 up/down: 0/-32) Total: -32 bytes
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'modutils/modutils.c')
| -rw-r--r-- | modutils/modutils.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/modutils/modutils.c b/modutils/modutils.c index 84300d9..ef4134a 100644 --- a/modutils/modutils.c +++ b/modutils/modutils.c @@ -190,6 +190,11 @@ int FAST_FUNC bb_delete_module(const char *module, unsigned int flags) return errno; } +/* Note: not suitable for delete_module() errnos. + * For them, probably only EWOULDBLOCK needs explaining: + * "Other modules depend on us". So far we don't do such + * translation and don't use moderror() for removal errors. + */ const char* FAST_FUNC moderror(int err) { switch (err) { |