Bug 23090 - cifsmount - Segmentation fault
: cifsmount - Segmentation fault
Status: CLOSED FIXED
: Sisyphus
(All bugs in Sisyphus/samba-client)
: unstable
: all Linux
: P3 normal
Assigned To:
:
:
:
:
:
  Show dependency tree
 
Reported: 2010-03-08 00:10 by
Modified: 2010-05-29 01:23 (History)


Attachments
Патч для cifsmount (581 bytes, patch)
2010-03-08 00:12, Evgeny Sinelnikov
no flags Details | Diff


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2010-03-08 00:10:33
Заметил странное падение утилиты монтирования 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 From 2010-03-08 00:12:57 -------
Created an attachment (id=4292) [details]
Патч для cifsmount

Минимальное исправление для cifsmount
------- Comment #2 From 2010-03-08 07:46:50 -------
Я подумываю перейти на cifs-utils 4.0, которые Jeff вынес из общего дерева в
отдельный проект, вместо встроенной версии. Там эта ошибка уже исправлена.
------- Comment #3 From 2010-05-29 01:23:34 -------
Насколько я понимаю, в samba-3.4.8, попавшей в Сизиф, этот баг исправлен.