gdb: Fix "info os <unknown>" command

Running `info os someUnknownOsType` is crashing when gdb is built with
-D_GLIBCXX_DEBUG:

	/usr/include/c++/5/debug/vector:439:error: attempt to
	access an element in an empty container.

In target_read_stralloc from target.c, the call to
target_read_alloc_1 can return an empty vector, we then call vector::back on
this vector, which is invalid.

This commit adds a check for emptiness before trying to call
vector::back on it. It also adds test to check for `info os <unknown>`
to return the proper error message.

This is a regression in gdb 8.2 and this patch restores the behavior of
previous versions.

gdb/ChangeLog:

	PR gdb/23974
	* target.c (target_read_stralloc): Check for empty vector.

gdb/testsuite/ChangeLog:

	PR gdb/23974
	* gdb.base/info-os.exp: Check return for unknown "info os" type.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 4ead538e674b2fa91141b537a1f0552583fa2f4b..db7a745a9cd7da4fc5c6108799b2aa2d12ce629e 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2018-12-21  Paul Marechal  <paul.marechal@ericsson.com>
+
+       PR gdb/23974
+       * target.c (target_read_stralloc): Check for empty vector.
+
 2018-12-21  Дилян Палаузов  <dilyan.palauzov@aegee.org>
 
        * top.c (print_gdb_configuration): Print "--without-python"

diff --git a/gdb/target.c b/gdb/target.c
index 80b8453176478f09f537be5c7ce4edad5a7ad15c..6c63255d03c4221b223da4d16933b534cfee96c6 100644 (file)
--- a/gdb/target.c
+++ b/gdb/target.c
@@ -1793,7 +1793,7 @@ target_read_stralloc (struct target_ops *ops, enum target_object object,
   if (!buf)
     return {};
 
-  if (buf->back () != '\0')
+  if (buf->empty () || buf->back () != '\0')
     buf->push_back ('\0');
 
   /* Check for embedded NUL bytes; but allow trailing NULs.  */

diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog
index c499d624dc4a68665435924b4b449009be2b5acd..f3224d4b4e41c4b4bca3fac50d24f8c9b32d37e0 100644 (file)
--- a/gdb/testsuite/ChangeLog
+++ b/gdb/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2018-12-21  Paul Marechal  <paul.marechal@ericsson.com>
+
+       PR gdb/23974
+       * gdb.base/info-os.exp: Check return for unknown "info os" type.
+
 2018-12-20  Philippe Waroquiers  <philippe.waroquiers@skynet.be>
 
        * gdb.ada/info_auto_lang.exp: Normalize some_c source file.

diff --git a/gdb/testsuite/gdb.base/info-os.exp b/gdb/testsuite/gdb.base/info-os.exp
index 8d7eab9c530b6ff945032b7efa2048d5b212a41d..c44e6b27c38919b8b4010e98dc712c70a1ed1235 100644 (file)
--- a/gdb/testsuite/gdb.base/info-os.exp
+++ b/gdb/testsuite/gdb.base/info-os.exp
@@ -175,6 +175,9 @@ expect_multiline "info os semaphores" "$semkey +$semid +666 +1 .*" "get semaphor
 #                               key   msqid   perm  num used bytes  num messages  last msgsnd() command  last msgrcv() command  user  group  creator user  creator group  last msgsnd() time  last msgrcv() time  last msgctl() time
 expect_multiline "info os msg" "$msgkey +$msqid +666 .*" "get message queues"
 
+gdb_test "info os unknown_entry" [multi_line \
+       "warning: Empty data returned by target.  Wrong osdata type\\\?" \
+       "Can not fetch data now."]
 
 # The SysV IPC primitives linger on after the creating process is killed
 # unless they are destroyed explicitly, so allow the test program to tidy