[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: extensions for emacsclient (CVS version)
From: |
Andreas Büsching |
Subject: |
Re: extensions for emacsclient (CVS version) |
Date: |
Wed, 10 Sep 2003 11:00:36 +0200 |
User-agent: |
Gnus/5.1003 (Gnus v5.10.3) Emacs/21.3.50 (gnu/linux) |
Hi,
Miles Bader <address@hidden> wrote:
> Andreas Büsching <address@hidden> writes:
>> - sprintf (server.sun_path, "/tmp/emacs%d-%s/server", (int) geteuid (),
>> system_name);
>> + sprintf (server.sun_path, "%s", socket_name);
>> + sprintf (server.sun_path, "/tmp/emacs%d-%s/server", (int) geteuid (),
>> system_name);
>
> Hmmm, no bounds checking when writing into server.sun_path (the old code
> didn't either, but the constructed path is probably less likely to
> overflow).
If this is the only problem that is simple to fix! So we will replace
'sprintf(' with 'snprintf( sizeof server.sun_path, '. Is that what you
mean?
crunchy
--
Hard work never killed anyone, but why give it a chance?
- extensions for emacsclient (CVS version), Andreas Büsching, 2003/09/10
- Re: extensions for emacsclient (CVS version), Miles Bader, 2003/09/10
- Re: extensions for emacsclient (CVS version),
Andreas Büsching <=
- Re: extensions for emacsclient (CVS version), Miles Bader, 2003/09/10
- Re: extensions for emacsclient (CVS version), Andreas Büsching, 2003/09/10
- Re: extensions for emacsclient (CVS version), Miles Bader, 2003/09/10
- Re: extensions for emacsclient (CVS version), Andreas Büsching, 2003/09/10
- Re: extensions for emacsclient (CVS version), Stefan Monnier, 2003/09/10
- Re: extensions for emacsclient (CVS version), Andreas Büsching, 2003/09/10
- Re: extensions for emacsclient (CVS version), Stefan Monnier, 2003/09/10
- Re: extensions for emacsclient (CVS version), Andreas Büsching, 2003/09/10
- Re: extensions for emacsclient (CVS version), Stefan Monnier, 2003/09/10
- Re: extensions for emacsclient (CVS version), David Kastrup, 2003/09/10