On Tue, Jun 10, 2014 at 7:32 PM, Dan Albert <[email protected]> wrote:
> mktemp was removed from POSIX 2008 for security reasons.
> ---
>  usr.bin/nc/netcat.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/usr.bin/nc/netcat.c b/usr.bin/nc/netcat.c
> index a034bba..d9e4ab4 100644
> --- a/usr.bin/nc/netcat.c
> +++ b/usr.bin/nc/netcat.c
> @@ -283,8 +283,8 @@ main(int argc, char *argv[])
>                 } else {
>                         strlcpy(unix_dg_tmp_socket_buf, "/tmp/nc.XXXXXXXXXX",
>                                 UNIX_DG_TMP_SOCKET_SIZE);
> -                       if (mktemp(unix_dg_tmp_socket_buf) == NULL)
> -                               err(1, "mktemp");
> +                       if (mkstemp(unix_dg_tmp_socket_buf) == NULL)

Doesn't mkstemp return a file descriptor, not a pointer?

> +                               err(1, "mkstemp");
>                         unix_dg_tmp_socket = unix_dg_tmp_socket_buf;
>                 }
>         }
> --
> 2.0.0.526.g5318336
>

Reply via email to