Re: Compiling C++ modules

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 4/24/06, Martin Mares <[email protected]> wrote:
> Hello!
>
> > Oh, and yeah, a = b + c *is* more readable than
> >
> > a = malloc(strlen(b) + strlen(c));
> > strcpy(a,b);
> > strcat(a,c);
> >
> > and contains fewer bugs ;)
>
> Actually, it contains at least the bug you have made in your C example,
> that is forgetting that malloc() can fail. So can string addition, if
> allocated dynamically.

It's too small of a fragment to tell whether or not appropriate
exception handling has been set up, but yeah, it needs a try/catch to
be safe.  That's *1* of the bugs in the c example.  It's the one they
share.

It's not the only one in the C code, though, as Willy Tarreau pointed
out, the malloc idiom is wrong, since it doesn't allocate space for
the terminating null.

Of course, the C fragment has the implicit problem of who will do the
associated free to avoid the memory leak, where the C++ fragment has
the issue of garbage collection...
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux