diff options
| author | Takashi Iwai <tiwai@suse.de> | 2013-10-28 12:54:52 +0100 |
|---|---|---|
| committer | Takashi Iwai <tiwai@suse.de> | 2013-10-29 11:37:42 +0100 |
| commit | 60f6fef877d52525b8887d27cfacf74a279e3e12 (patch) | |
| tree | bf3744c3967ff8fbdcb24a6eb1e88685c5078b2c /sound/core | |
| parent | 1f96153bc613c526f5ea39a7f366b3bdf70249be (diff) | |
| download | linux-60f6fef877d52525b8887d27cfacf74a279e3e12.tar.gz linux-60f6fef877d52525b8887d27cfacf74a279e3e12.tar.bz2 linux-60f6fef877d52525b8887d27cfacf74a279e3e12.zip | |
ALSA: Optimize module name check
module->name is a fixed array, so we can check the empty contents
straightforwardly in module_slot_match().
Spotted by coverity CID 1056786.
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/core')
| -rw-r--r-- | sound/core/init.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/core/init.c b/sound/core/init.c index 6b9087115da2..01a89383a062 100644 --- a/sound/core/init.c +++ b/sound/core/init.c @@ -66,7 +66,7 @@ static int module_slot_match(struct module *module, int idx) #ifdef MODULE const char *s1, *s2; - if (!module || !module->name || !slots[idx]) + if (!module || !*module->name || !slots[idx]) return 0; s1 = module->name; |