[xentronium]

strcmp (and many other string routines like strlen, strcpy) relies on strings being null terminated. Safe versions (strncmp, strnlen, strncpy) have a parameter for maximum string length.

I am pretty sure that in this particular case it is not passed any user input, so it's kinda safe.

[Locke1689]

In this case it should be fine because the second argument is a string literal (guaranteed to be null-terminated) and the function doesn't continue past the end of the first null-terminated string.

[antihero]

Would be a pretty neat exploit to buffer overflow someone's actual kernel via a carefully constructed module name, though.

[1amzave]

A neat hack, yes...but probably not worthwhile from the perspective of an actual attacker (if you've got permissions to load a kernel module, you could just load one of your own crafting to do whatever nefarious things you wanted directly).

[ori_b]

If you haven't validated or ensured that your string is a string you've already lost.