On 19 Jul 2003 22:58:57 +1000
Sven Dowideit <svenud@ozemail.com.au> wrote:
> @@ -4838,7 +4850,7 @@
> readCapabilityRid(local, &cap_rid);
>
> dwrq->length = sizeof(struct iw_range);
> - memset(range, 0, sizeof(*range));
> + memset(range, 0, sizeof(range));
> range->min_nwid = 0x0000;
> range->max_nwid = 0x0000;
> range->num_channels = 14;
I suspect that this part of the patch to airo.c is incorrect. The
intent seems to be to clear a section of memory pointed to by range that
contains (or will soon contain) a struct iw_range. The sizeof(*range)
is equivalent of the sizeof(struct iw_range) above. The patch reduces
the size of the memset to the size of the pointer (which I'm assuming is
smaller than the structure [/me goes and looks]).
Of course, the range pointer is derived from the char *extra
parameter... this could mean that we're actually getting a pre-filled
iw_range and the memset is only supposed to clear the first member. If
that's the case, I would hope that the author could come up with a
cleaner way of expressing that.
-g
This archive was generated by hypermail 2b29 : Wed Jul 23 2003 - 22:00:43 EST