Re: [devel] [git update] packages/ppp: heads/master

Re: [devel] [git update] packages/ppp: heads/master

[Slava Semushin]

2007/12/8, Denis Smirnov <mithraen / altlinux.org>:
> Update of /people/mithraen/packages/ppp.git
[...]
> +--- a/ppp/pppd/plugins/radius/clientid.c
> ++++ b/ppp/pppd/plugins/radius/clientid.c
> +@@ -104,13 +104,13 @@ int rc_read_mapfile(char *filename)
> + UINT4 rc_map2id(char *name)
> + {
> +       struct map2id_s *p;
> +-      char ttyname[PATH_MAX];
> ++      char ttyname[PATH_MAX + 1];
> +
> +       *ttyname = '\0';
> +       if (*name != '/')
> +               strcpy(ttyname, "/dev/");
> +
> +-      strncat(ttyname, name, sizeof(ttyname));
> ++      strlcat(ttyname, name, sizeof(ttyname));
> +
> +       for(p = map2id_list; p; p = p->next)
> +               if (!strcmp(ttyname, p->name)) return p->id;
[...]

Так а зачем размер ttyname было увеличивать на один? Насколько
понимаю, это совсем не обязательно.


-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru

Re: [devel] [git update] packages/ppp: heads/master

[Hihin Ruslan]

[-- Attachment #1: Type: text/plain, Size: 573 bytes --]

Здравствуйте Slava Semushin
  В сообщении от 10 декабря 2007 Slava Semushin написал(a):
 > Так а зачем размер ttyname было увеличивать на один? Насколько
 >
 > понимаю, это совсем не обязательно.

Я не Вас понял - если в имени максимум PATH_MAX символов, то надо 
добавить ещё место для \0, иначе эта строка может слиться, например, со 
следующей.

-- 
  А ещё говорят так  (fortune):
 
- Что делает системный администратор, проснувшись с похмелья? - 
Тестирует память...  

________________________________________________________________________
С уважением Хихин Руслан

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] [git update] packages/ppp: heads/master

[Slava Semushin]

10.12.07, Hihin Ruslan<ruslandh / gmail.com> написал(а):
[...]
> Я не Вас понял - если в имени максимум PATH_MAX символов, то надо
> добавить ещё место для \0, иначе эта строка может слиться, например, со
> следующей.

Это требуется в случае использования strncat. Для strlcat этого не
требуется, если я правильно понял написанное в мане.


-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru

Re: [devel] [git update] packages/ppp: heads/master

[Денис Смирнов]

[-- Attachment #1: Type: text/plain, Size: 782 bytes --]

On Tue, Dec 11, 2007 at 01:48:51PM +0600, Slava Semushin wrote:

>> Я не Вас понял - если в имени максимум PATH_MAX символов, то надо
>> добавить ещё место для \0, иначе эта строка может слиться, например, со
>> следующей.
SS> Это требуется в случае использования strncat. Для strlcat этого не
SS> требуется, если я правильно понял написанное в мане.

Я до конца не понял, а так как ответа в листе не получил --
перестраховался.

Кроме того, я не совсем понимаю смысл константы PATH_MAX -- учитывает она
завершающий \0 в своем размере или нет.

-- 
С уважением, Денис

http://freesource.info
----------------------------------------------------------------------------
> В системе установлен kudzu ?
Конечно да. Какое хайку без кудзы? :)
		-- avp in sisyphus@

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] [git update] packages/ppp: heads/master

[Slava Semushin]

11.12.07, Денис Смирнов<mithraen / altlinux.ru> написал(а):
[...]
> Кроме того, я не совсем понимаю смысл константы PATH_MAX -- учитывает она
> завершающий \0 в своем размере или нет.

[c0der@rock ~]$ grep -r PATH_MAX
/usr/include/linux-default/include/linux/limits.h
#define PATH_MAX        4096    /* # chars in a path name including nul */

Но тут ещё зависит от используемой ф-ции, насколько могу судить.

-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru

Re: [devel] [git update] packages/ppp: heads/master

[Хихин Руслан]

В сообщении от Tuesday 11 December 2007 16:24:19 Денис Смирнов написал(а):
> Кроме того, я не совсем понимаю смысл константы PATH_MAX -- учитывает она
> завершающий \0 в своем размере или нет.

Не учитывает.

-- 
С Уважением Хихин Руслан.