diff options
| author | Greg Kaiser <gkaiser@google.com> | 2017-09-05 15:55:41 -0700 |
|---|---|---|
| committer | Moyster <oysterized@gmail.com> | 2017-09-14 18:53:40 +0200 |
| commit | 9fb5a02145a2b3e39a60fca0983ddd525b617f95 (patch) | |
| tree | 92acf6fcb7ed19e2c1e9269a4d9f5799e714bc1f | |
| parent | 3927d76c2da12e5353580ae22c664d4c5c6959a0 (diff) | |
ANDROID: fiq_debugger: Fix minor bug in code
We fix a typo in the code which had us comparing a pointer instead
of the value which was being pointed to. This turns out to be
a relatively benign bug, as we'd incorrectly pass in the empty
string instead of NULL to the function, and the function can handle
both. But we fix it so the code is clearly doing what we intend.
Signed-off-by: Greg Kaiser <gkaiser@google.com>
Change-Id: Ib059819775a3bebca357d4ce684be779853156e3
| -rw-r--r-- | drivers/staging/android/fiq_debugger/fiq_debugger.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/android/fiq_debugger/fiq_debugger.c b/drivers/staging/android/fiq_debugger/fiq_debugger.c index ceb45bc9e..0794b3e62 100644 --- a/drivers/staging/android/fiq_debugger/fiq_debugger.c +++ b/drivers/staging/android/fiq_debugger/fiq_debugger.c @@ -396,7 +396,7 @@ static void fiq_debugger_work(struct work_struct *work) cmd += 6; while (*cmd == ' ') cmd++; - if ((cmd != '\0') && sysrq_on()) + if ((*cmd != '\0') && sysrq_on()) kernel_restart(cmd); else kernel_restart(NULL); |