aboutsummaryrefslogtreecommitdiff
path: root/modutils/modprobe.c
diff options
context:
space:
mode:
authorDenis Vlasenko <vda.linux@googlemail.com>2008-05-22 17:37:38 +0000
committerDenis Vlasenko <vda.linux@googlemail.com>2008-05-22 17:37:38 +0000
commitae84b11467c56316a43de6146100ce22f24cf622 (patch)
tree283fd8ebb6804e1f1caefb2431ce001690d45b87 /modutils/modprobe.c
parentdfd8282464c56eda0dba62ad3b9a2a4b71ba725f (diff)
downloadbusybox-ae84b11467c56316a43de6146100ce22f24cf622.tar.gz
modprobe: fix for blacklisting
The patch makes the order of the alias/blacklist in modprobe.conf irrelevant (like module-utils' modprobe). In first patch the alias had to be defined before the blacklist which caused problems because the modprobe.conf file is read before /lib/modules/*/modules.alias. The attatched patch will mark the blacklisted module itself rather than trying to find the alias that points to the blacklisted module and test this flag later in the alias resolving stage.
Diffstat (limited to 'modutils/modprobe.c')
-rw-r--r--modutils/modprobe.c9
1 files changed, 5 insertions, 4 deletions
diff --git a/modutils/modprobe.c b/modutils/modprobe.c
index 60dc92665..b7903469d 100644
--- a/modutils/modprobe.c
+++ b/modutils/modprobe.c
@@ -332,7 +332,7 @@ static void include_conf(struct dep_t **first, struct dep_t **current, char *buf
mod = skip_whitespace(buffer + 10);
for (dt = *first; dt; dt = dt->m_next) {
- if (dt->m_isalias && strcmp(dt->m_deparr[0], mod) == 0)
+ if (strcmp(dt->m_name, mod) == 0)
break;
}
if (dt)
@@ -748,12 +748,13 @@ static void check_dep(char *mod, struct mod_list_t **head, struct mod_list_t **t
// resolve alias names
while (dt->m_isalias) {
- if (dt->m_depcnt == 1 && !(ENABLE_FEATURE_MODPROBE_BLACKLIST &&
- dt->m_isblacklisted)) {
+ if (dt->m_depcnt == 1) {
struct dep_t *adt;
for (adt = depend; adt; adt = adt->m_next) {
- if (check_pattern(adt->m_name, dt->m_deparr[0]) == 0)
+ if (check_pattern(adt->m_name, dt->m_deparr[0]) == 0 &&
+ !(ENABLE_FEATURE_MODPROBE_BLACKLIST &&
+ adt->m_isblacklisted))
break;
}
if (adt) {