diff options
| author | Tom de Vries <tdevries@suse.de> | 2025-03-31 16:12:22 +0200 |
|---|---|---|
| committer | Tom de Vries <tdevries@suse.de> | 2025-03-31 16:12:22 +0200 |
| commit | 80a7eb6ac7d6661b06e014329cac101e83d2003c (patch) | |
| tree | 8a0c0e7598009b9530d432bf32c19b2592228db3 | |
| parent | 7b34a95034ff62537db66049a1736e7b4436e499 (diff) | |
| download | binutils-80a7eb6ac7d6661b06e014329cac101e83d2003c.zip binutils-80a7eb6ac7d6661b06e014329cac101e83d2003c.tar.gz binutils-80a7eb6ac7d6661b06e014329cac101e83d2003c.tar.bz2 | |
[gdb] Check strpbrk against nullptr
In noticed two occurrences of "if (strpbrk (...))".
Fix this style issue by checking against nullptr.
| -rw-r--r-- | gdb/go32-nat.c | 2 | ||||
| -rw-r--r-- | gdbsupport/common-inferior.cc | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/gdb/go32-nat.c b/gdb/go32-nat.c index 8453366..7852f56 100644 --- a/gdb/go32-nat.c +++ b/gdb/go32-nat.c @@ -697,7 +697,7 @@ go32_nat_target::create_inferior (const char *exec_file, "not enough memory.\n")); /* Parse the command line and create redirections. */ - if (strpbrk (args, "<>")) + if (strpbrk (args, "<>") != nullptr) { if (redir_cmdline_parse (args, &child_cmd) == 0) args = child_cmd.command; diff --git a/gdbsupport/common-inferior.cc b/gdbsupport/common-inferior.cc index e544780..d2fd348 100644 --- a/gdbsupport/common-inferior.cc +++ b/gdbsupport/common-inferior.cc @@ -59,7 +59,7 @@ escape_characters (const char *arg, const char *special) #ifdef __MINGW32__ bool quoted = false; - if (strpbrk (arg, special)) + if (strpbrk (arg, special) != nullptr) { quoted = true; result += quote; |