diff options
| author | Andreas Schwab <schwab@linux-m68k.org> | 2004-10-23 19:13:26 +0000 |
|---|---|---|
| committer | Andreas Schwab <schwab@linux-m68k.org> | 2004-10-23 19:13:26 +0000 |
| commit | f27cf670a8729e84799f86c25401fe1be5cd2e51 (patch) | |
| tree | b545725767b1783c3ee8d65e2a6d711e1a0debbb | |
| parent | 0fd555c412ecfb816f441dce78a3f288c49eb8e5 (diff) | |
| download | fsf-binutils-gdb-f27cf670a8729e84799f86c25401fe1be5cd2e51.zip fsf-binutils-gdb-f27cf670a8729e84799f86c25401fe1be5cd2e51.tar.gz fsf-binutils-gdb-f27cf670a8729e84799f86c25401fe1be5cd2e51.tar.bz2 | |
* ada-lang.c (grow_vect): Return the new array instead of
expecting pointer to pointer to avoid violation of aliasing rules.
* ada-lang.h (GROW_VECT): Adapt call to grow_vect.
| -rw-r--r-- | gdb/ChangeLog | 6 | ||||
| -rw-r--r-- | gdb/ada-lang.c | 11 | ||||
| -rw-r--r-- | gdb/ada-lang.h | 4 |
3 files changed, 14 insertions, 7 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 12485fa..0df3786 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,9 @@ +2004-10-23 Andreas Schwab <schwab@suse.de> + + * ada-lang.c (grow_vect): Return the new array instead of + expecting pointer to pointer to avoid violation of aliasing rules. + * ada-lang.h (GROW_VECT): Adapt call to grow_vect. + 2004-10-23 Ulrich Weigand <uweigand@de.ibm.com> * s390-tdep.c (enum pv_boolean): Remove. diff --git a/gdb/ada-lang.c b/gdb/ada-lang.c index 275e3a0..00e61a4 100644 --- a/gdb/ada-lang.c +++ b/gdb/ada-lang.c @@ -319,20 +319,21 @@ extract_string (CORE_ADDR addr, char *buf) while (buf[char_index - 1] != '\000'); } -/* Assuming *OLD_VECT points to an array of *SIZE objects of size +/* Assuming VECT points to an array of *SIZE objects of size ELEMENT_SIZE, grow it to contain at least MIN_SIZE objects, - updating *OLD_VECT and *SIZE as necessary. */ + updating *SIZE as necessary and returning the (new) array. */ -void -grow_vect (void **old_vect, size_t * size, size_t min_size, int element_size) +void * +grow_vect (void *vect, size_t *size, size_t min_size, int element_size) { if (*size < min_size) { *size *= 2; if (*size < min_size) *size = min_size; - *old_vect = xrealloc (*old_vect, *size * element_size); + vect = xrealloc (vect, *size * element_size); } + return vect; } /* True (non-zero) iff TARGET matches FIELD_NAME up to any trailing diff --git a/gdb/ada-lang.h b/gdb/ada-lang.h index 18a662a..3df4aa4 100644 --- a/gdb/ada-lang.h +++ b/gdb/ada-lang.h @@ -171,9 +171,9 @@ extern struct task_entry *task_list; least M objects, updating V and S as necessary. */ #define GROW_VECT(v, s, m) \ - if ((s) < (m)) grow_vect ((void**) &(v), &(s), (m), sizeof(*(v))); + if ((s) < (m)) (v) = grow_vect (v, &(s), m, sizeof *(v)); -extern void grow_vect (void **, size_t *, size_t, int); +extern void *grow_vect (void *, size_t *, size_t, int); extern int ada_get_field_index (const struct type *type, const char *field_name, |