cscope / Patches / #52 execl terminates with NULL

#52 execl terminates with NULL

Status: closed-fixed

Owner: Hans-Bernhard Broeker

Labels: None

Priority: 5

Updated: 2005-01-30

Created: 2005-01-29

Creator: Marcus Meissner

Private: No

Hi,

the execlp argument list needs to be terminated with NULL.
(0 is not NULL in the va_list case)

Ciao, Marcus

Discussion

Marcus Meissner - 2005-01-29

cscope-null.patch

cscope-null.patch

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Hans-Bernhard Broeker - 2005-01-30

assigned_to: nobody --> broeker

status: open --> closed-fixed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Hans-Bernhard Broeker - 2005-01-30

Logged In: YES
user_id=27517

Thanks for spotting this.

Actually, IIRC, even NULL isn't really safe --- C compilers
can legally #define NULL 0. The pointer context has to be
established at the point of usage, which means only a
full-fledged (void *)0 will do.

Change going into CVS right away.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Marcus Meissner - 2005-01-30

Logged In: YES
user_id=48092

question is also why doesnt it use the
libc popen on Linux.

Ciao, Marcus

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Hans-Bernhard Broeker - 2005-01-30

Logged In: YES
user_id=27517

No idea. Please keep in mind that this is code whose first
version predates Linus' birth. I.e. it's probably just
"hysterical raisins" ;-)

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.