[LTP] [RFC PATCH 2/9] lib: Add a canary for guarded buffers
Cyril Hrubis
chrubis@suse.cz
Thu Aug 8 11:27:32 CEST 2019
Hi!
> > +static void check_canary(struct map *map)
> > +{
> > + size_t i;
> > + char *buf = map->addr;
> > +
> > + for (i = 0; i < map->buf_shift/2; i++) {
> > + if (buf[map->buf_shift - i - 1] != buf[i]) {
> > + tst_res(TWARN,
> > + "pid %i: buffer modified before address %p %zu",
> > + (char*)map->addr + map->buf_shift, i);
>
> Here you probably missed the getpid() for printing %i match :).
Ah, right, I guess that this is unfinished last minute modification.
Actually as it is the canaries are checked only for the main library pid
becuase the tst_free_all() is hooked up in the do_test_cleanup().
I guess that we should call it in the run_tests() function before we
call exit(0) for the child processes as well as in the tst_vbrk_() and
after that changes printing pid in the warning makes sense.
--
Cyril Hrubis
chrubis@suse.cz
More information about the ltp
mailing list