Bug 23090

Summary:

cifsmount - Segmentation fault

Product:

Sisyphus

Reporter:

Evgeny Sinelnikov <sin>

Component:

samba-client

Assignee:

Evgeny Sinelnikov <sin>

Status:

CLOSED FIXED

QA Contact:

qa-sisyphus

Severity:

normal

Priority:

CC:

sin

Version:

unstable

Hardware:

all

OS:

Linux

Attachments:

Description	Flags
Патч для cifsmount	none

Description Evgeny Sinelnikov 2010-03-08 00:10:33 MSK

Заметил странное падение утилиты монтирования cifs-ресурсов
$ cifsmount
Segmentation fault

Нашёл, как минимум одну часть некорректного кода:
--- a/samba/source/client/mount.cifs.c
+++ b/samba/source/client/mount.cifs.c
@@ -1098,8 +1098,8 @@ int main(int argc, char ** argv)
                }
                mountpoint = argv[2];
        } else {
-               if ((strcmp (argv[1], "--version") == 0) ||
-                   ((strcmp (argv[1], "-V") == 0))) {
+               if (argv[1] && ((strcmp (argv[1], "--version") == 0) ||
+                   ((strcmp (argv[1], "-V") == 0)))) {
                        printf ("mount.cifs version: %s.%s%s\n",
                        MOUNT_CIFS_VERSION_MAJOR,
                        MOUNT_CIFS_VERSION_MINOR,

Comment 1 Evgeny Sinelnikov 2010-03-08 00:12:57 MSK

Created attachment 4292 [details]
Патч для cifsmount

Минимальное исправление для cifsmount

Comment 2 Alexander Bokovoy 2010-03-08 07:46:50 MSK

Я подумываю перейти на cifs-utils 4.0, которые Jeff вынес из общего дерева в отдельный проект, вместо встроенной версии. Там эта ошибка уже исправлена.

Comment 3 Vitaly Kuznetsov 2010-05-29 01:23:34 MSD

Насколько я понимаю, в samba-3.4.8, попавшей в Сизиф, этот баг исправлен.