binutils-gdb/libctf/testsuite/libctf-regression/pptrtab.c
Nick Alcock e05a3e5a49 libctf: lookup_by_name: do not return success for nonexistent pointer types
The recent work allowing lookups of pointers in child dicts when the
pointed-to type is in the parent dict broke the case where a pointer
type that does not exist at all is looked up: we mistakenly return the
pointed-to type, which is likely not a pointer at all.  This causes
considerable confusion.

Fixed, with a new testcase.

libctf/ChangeLog
2021-01-19  Nick Alcock  <nick.alcock@oracle.com>

	* ctf-lookup.c (ctf_lookup_by_name_internal): Do not return the
	base type if looking up a nonexistent pointer type.
	* testsuite/libctf-regression/pptrtab*: Test it.
2021-01-19 12:45:19 +00:00

62 lines
1.4 KiB
C

#include <ctf-api.h>
#include <stdio.h>
#include <stdlib.h>
int
main (int argc, char *argv[])
{
ctf_dict_t *fp;
ctf_archive_t *ctf;
ctf_next_t *i = NULL;
ctf_id_t type;
const char *arcname;
char *type_name;
int err;
if (argc != 2)
{
fprintf (stderr, "Syntax: %s PROGRAM\n", argv[0]);
exit(1);
}
if ((ctf = ctf_open (argv[1], NULL, &err)) == NULL)
goto open_err;
/* Make sure we can look up a_t * by name in all non-parent dicts, even though
the a_t * and the type it points to are in distinct dicts; make sure we
cannot look up b_t *. */
while ((fp = ctf_archive_next (ctf, &i, &arcname, 1, &err)) != NULL)
{
if ((type = ctf_lookup_by_name (fp, "a_t *")) == CTF_ERR)
goto err;
if ((ctf_lookup_by_name (fp, "b_t *")) != CTF_ERR ||
ctf_errno (fp) != ECTF_NOTYPE)
goto noerr;
if (ctf_type_reference (fp, type) == CTF_ERR)
goto err;
printf ("%s: a_t * points to a type of kind %i\n", arcname,
ctf_type_kind (fp, ctf_type_reference (fp, type)));
ctf_dict_close (fp);
}
if (err != ECTF_NEXT_END)
goto open_err;
ctf_close (ctf);
return 0;
open_err:
fprintf (stderr, "%s: cannot open: %s\n", argv[0], ctf_errmsg (err));
return 1;
err:
fprintf (stderr, "Lookup failed in %s: %s\n", arcname, ctf_errmsg (ctf_errno (fp)));
return 1;
noerr:
fprintf (stderr, "Lookup unexpectedly succeeded in %s\n", arcname);
return 1;
}