Bug 23090 - cifsmount - Segmentation fault
Summary: cifsmount - Segmentation fault
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: samba-client (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: Evgeny Sinelnikov
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-03-08 00:10 MSK by Evgeny Sinelnikov
Modified: 2010-05-29 01:23 MSD (History)
1 user (show)

See Also:

Attachments
Патч для cifsmount (581 bytes, patch)
2010-03-08 00:12 MSK, Evgeny Sinelnikov 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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, попавшей в Сизиф, этот баг исправлен.