On Mon 04 Aug 2014 08:35:02 [email protected] wrote: > From: Zubin Mithra <[email protected]> > > * desc.c (decode_select): Initialize fds to NULL. > * strace.c (expand_tcbtab): Change type to unsigned. > (startup_child): Initialize pathname array.
you need to describe why you're making a change, not just how
> --- a/desc.c
> +++ b/desc.c
> @@ -314,7 +314,7 @@ decode_select(struct tcb *tcp, long *args, enum
> bitness_t bitness) {
> int i, j;
> int nfds, fdsize;
> - fd_set *fds;
> + fd_set *fds = NULL;
> const char *sep;
> long arg;
why ? i guess you're handling the case where nfds==0 ? seems like fds is
used uninitialized in that case.
however, if you do this, you'll see that the nfds<0 check also sets fds to
NULL, so you should delete that line.
> --- a/strace.c
> +++ b/strace.c
> @@ -675,7 +675,7 @@ expand_tcbtab(void)
> callers have pointers and it would be a pain.
> So tcbtab is a table of pointers. Since we never
> free the TCBs, we allocate a single chunk of many. */
> - int i = tcbtabsize;
> + unsigned int i = tcbtabsize;
i guess this is because tcpbtabsize is unsigned, and i is only used to compare
to that.
> @@ -1160,7 +1160,7 @@ startup_child(char **argv)
> {
> struct_stat statbuf;
> const char *filename;
> - char pathname[MAXPATHLEN];
> + char pathname[MAXPATHLEN] = "";
> int pid;
> struct tcb *tcp;
i'm not seeing a problem here. pathname looks like it's always initialized
before it's used. note that sizeof(pathname) is not using the value of
pathname, so those checks don't count ...
-mike
signature.asc
Description: This is a digitally signed message part.
------------------------------------------------------------------------------ Infragistics Professional Build stunning WinForms apps today! Reboot your WinForms applications with our WinForms controls. Build a bridge from your legacy apps to the future. http://pubads.g.doubleclick.net/gampad/clk?id=153845071&iu=/4140/ostg.clktrk
_______________________________________________ Strace-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/strace-devel
