mirrors/scst
1
0
Fork 0
mirror of https://github.com/SCST-project/scst.git synced 2026-05-21 12:41:26 +00:00
Code Issues Packages Projects Releases Wiki Activity
8,786 Commits 26 Branches 11 Tags
debug
Commit Graph

8786 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
Vladislav Bolkhovitin
2331e8ec33 Thre is potential buffer overflow in iscsi_session_alloc() due to 
short computation of needed string size.  Notice the "%s@%s" in the first call
to sprintf().

Signed-off-by: David Butterfield <dab21774@gmail.com>



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@7110 d57e44dd-8a1f-0410-8b47-8ef2f437770f
 2017-04-13 23:02:18 +00:00
Vladislav Bolkhovitin
aa6c6a4c2f Change memcpy() to strncpy() because the source name string is not guaranteed 
to exist as valid addressable memory beyond the NULL byte.

Signed-off-by: David Butterfield <dab21774@gmail.com>

with small addition to force set last byte NULL



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@7109 d57e44dd-8a1f-0410-8b47-8ef2f437770f
 2017-04-13 22:38:38 +00:00
Vladislav Bolkhovitin
d236725cae create_and_open_dev() returns a (-errno), so the "if (iser_fd...)" check should 
detect *any* negative return value as a case when fd should be set to -1.

Signed-off-by: David Butterfield <dab21774@gmail.com>



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@7108 d57e44dd-8a1f-0410-8b47-8ef2f437770f
 2017-04-13 22:30:25 +00:00
Vladislav Bolkhovitin
a23e34fb62 The argument to sleep() would get "promoted" to an integer with value zero. 
-               sleep(0.1);
+               usleep(100*1000);

Signed-off-by: David Butterfield <dab21774@gmail.com>



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@7107 d57e44dd-8a1f-0410-8b47-8ef2f437770f
 2017-04-13 22:04:07 +00:00
Bart Van Assche
1a035743eb Merge branch 'svn-trunk' 2017-04-11 21:14:11 -07:00
Bart Van Assche
9529f5e5b3 ib_srpt: Ensure that the BUG_ON() argument has no side effects 
Reported-by: David Butterfield <dab21774@gmail.com>


git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@7106 d57e44dd-8a1f-0410-8b47-8ef2f437770f
 2017-04-11 18:48:36 +00:00
Bart Van Assche
624c72997b Merge branch 'svn-trunk' 2017-03-07 20:59:24 -08:00
Bart Van Assche
76daca5677 scst_lib.c: Fix indentation 
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@7105 d57e44dd-8a1f-0410-8b47-8ef2f437770f
 2017-03-08 04:43:23 +00:00
Vladislav Bolkhovitin
d91dd9b3ed Linux kernel v4.10 build fix. 
Signed-off-by: Sebastian Herbszt <herbszt@gmx.de>



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@7104 d57e44dd-8a1f-0410-8b47-8ef2f437770f
 2017-03-08 04:36:45 +00:00
Bart Van Assche
7d452207c4 scst: Unbreak the kernel v2.6.38 build 2017-03-01 22:54:42 -08:00
Bart Van Assche
382cd6ada7 fcst: Port to Linux kernel v4.10 2017-03-01 22:35:42 -08:00
Bart Van Assche
5c32179226 nightly build: Update kernel configs 2017-03-01 22:24:25 -08:00
Bart Van Assche
c8b8d8a2f0 Merge branch 'svn-trunk' 2017-03-01 18:51:51 -08:00
Bart Van Assche
4481d52e8d scripts: Update kernel download location 2017-03-01 16:15:21 -08:00
Bart Van Assche
1bf5d439d6 scst: Coding style update related to the use of the keyword 'unsigned' 
Use 'unsigned int' instead of 'unsigned' and 'unsigned long long'
instead of 'long long unsigned'.


git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@7103 d57e44dd-8a1f-0410-8b47-8ef2f437770f
 2017-03-02 00:10:24 +00:00
Bart Van Assche
e372a0123e scst: Coding style updates related to whitespace 
This patch changes whitespace only and nothing else.



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@7102 d57e44dd-8a1f-0410-8b47-8ef2f437770f
 2017-03-02 00:09:14 +00:00
Bart Van Assche
ca0ad87e2a Merge branch 'svn-trunk' 2017-03-01 07:32:25 -08:00
Bart Van Assche
0bb6de9471 scst_vdisk: Avoid that LUN refresh triggers a general protection fault 
Avoid that triggering LUN referesh concurrently with device deletion
triggers the following:

general protection fault: 0000 [#1]
Workqueue: events vdev_inq_changed_fn [scst_vdisk]
Call Trace:
 _raw_spin_lock_bh+0x2b/0x30
 scst_cm_update_dev+0x87/0x190 [scst]
 scst_dev_inquiry_data_changed+0xfb/0x1b0 [scst]
 vdev_inq_changed_fn+0x60/0x120 [scst_vdisk]
 process_one_work+0x14d/0x410
 worker_thread+0x66/0x460
 kthread+0xdb/0x100
 ret_from_fork+0x3f/0x70

Reported-by: Jinpu Wang <jinpu.wang@profitbricks.com>
Tested-by: Jinpu Wang <jinpu.wang@profitbricks.com>


git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@7101 d57e44dd-8a1f-0410-8b47-8ef2f437770f
 2017-03-01 15:31:59 +00:00
Bart Van Assche
b13fd42124 nightly/conf/nightly.sendmail: Use mail if mutt is not available 2017-02-28 22:10:49 -08:00
Bart Van Assche
84a80e83fe nightly build: Update kernel versions 2017-02-28 22:04:16 -08:00
Bart Van Assche
aa69fe88a5 Rename srpt/src/backport.h into srpt/src/ib_backport.h 2017-02-28 21:47:59 -08:00
Bart Van Assche
682eaed880 scst: Remove scst_device.dev_unregistering again 
See also:
- commit 2e386ef7c1 ("Merge branch svn-trunk")
- commit 3fc775c75a / r6589 ("EXTENDED COPY support")
- commit 6c46b1a5a0 ("scst: Remove the dev_unregistering member variable")
 2017-02-28 21:33:59 -08:00
Bart Van Assche
e009996237 scstadmin: Fix regression tests 2017-02-28 21:33:58 -08:00
Bart Van Assche
ebdb340e3a scst: Port to Linux kernel v4.11 2017-02-28 21:33:58 -08:00
Bart Van Assche
5b5a8b2523 scst_copy_mgr: Remove two incorrect assert statements 2017-02-28 20:35:39 -08:00
Bart Van Assche
bb59822be2 Merge branch 'svn-trunk' 2017-02-28 17:41:54 -08:00
Bart Van Assche
08f2570761 scst_copy_mgr: Warn if LUN lookup fails 
It really is a bug if LUN lookup fails. 


git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@7100 d57e44dd-8a1f-0410-8b47-8ef2f437770f
 2017-02-27 22:36:33 +00:00
Bart Van Assche
5abc3cdd5f Merge branch 'svn-trunk' 2017-02-24 18:55:15 -08:00
Bart Van Assche
048b49c353 scst_copy_mgr: Fix iteration over sess_tgt_dev_list 2017-02-24 17:56:45 -08:00
Bart Van Assche
19212e4f29 scst_copy_mgr: Avoid that scst_cm_check_access_acg() triggers a kernel warning 
Avoid that the following kernel warning is triggered:

WARNING: CPU: 2 PID: 1761 at scst/src/scst_copy_mgr.c:2835 
Call Trace:
 dump_stack+0x86/0xc3
 __warn+0xc1/0xe0
 warn_slowpath_null+0x18/0x20
 scst_cm_check_access_acg+0x1c2/0x1d0 [scst]
 scst_cm_check_access+0x155/0x280 [scst]
 scst_cm_parse_id_tgt_descr+0x2e6/0x590 [scst]
 scst_cm_parse_descriptors+0x303/0xa60 [scst]
 scst_parse_descriptors+0x5b/0x130 [scst]
 scst_tgt_pre_exec+0x370/0x4a0 [scst]
 scst_process_active_cmd+0x455/0x770 [scst]
 scst_cmd_thread+0x1fb/0x690 [scst]
 kthread+0x102/0x140
 ret_from_fork+0x31/0x40



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@7099 d57e44dd-8a1f-0410-8b47-8ef2f437770f
 2017-02-25 01:56:30 +00:00
Bart Van Assche
7ad87045dc scst_copy_mgr: Avoid that scst_cm_on_add_lun() triggers a kernel warning 
Avoid that the following warning is triggered:

WARNING: CPU: 1 PID: 1282 at scst/src/scst_copy_mgr.c:2784 scst_cm_on_add_lun+0x141/0x1b0 [scst]
Call Trace:
 dump_stack+0x86/0xc3
 __warn+0xc1/0xe0
 warn_slowpath_null+0x18/0x20
 scst_cm_on_add_lun+0x141/0x1b0 [scst]
 scst_acg_add_lun+0x27f/0x370 [scst]
 __scst_process_luns_mgmt_store+0x589/0x5f0 [scst]
 scst_luns_mgmt_store_work_fn+0x22/0x30 [scst]
 scst_process_sysfs_works+0x7b/0x1c0 [scst]
 sysfs_work_thread_fn+0x11b/0x280 [scst]
 kthread+0x102/0x140
 ret_from_fork+0x31/0x40


git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@7098 d57e44dd-8a1f-0410-8b47-8ef2f437770f
 2017-02-25 01:54:58 +00:00
Bart Van Assche
425f4ab3f2 Merge branch 'svn-trunk' 2017-02-23 23:08:18 -08:00
Bart Van Assche
95fe66fbeb scst: Fix locking annotations 
Avoid that the following warnings are triggered:

WARNING: CPU: 5 PID: 32343 at scst/src/scst_copy_mgr.c:2741 scst_cm_on_add_acg+0x12d/0x180 [scst]
Call Trace:
 [<ffffffff812a4d23>] dump_stack+0x86/0xc3
 [<ffffffff8106412b>] __warn+0xcb/0xf0
 [<ffffffff8106421d>] warn_slowpath_null+0x1d/0x20
 [<ffffffffa04f9b0d>] scst_cm_on_add_acg+0x12d/0x180 [scst]
 [<ffffffffa04b8084>] scst_alloc_add_acg+0xc4/0x280 [scst]
 [<ffffffffa0486b30>] scst_register_target+0x190/0x3a0 [scst]
 [<ffffffffa054f2de>] scst_cm_init+0xb8/0xdda [scst]
 [<ffffffffa054eab9>] init_scst+0xa06/0xc3f [scst]
 [<ffffffff8100043d>] do_one_initcall+0x3d/0x160
 [<ffffffff81154ae0>] do_init_module+0x5f/0x1f1
 [<ffffffff810efa69>] load_module+0x2719/0x2a50
 [<ffffffff810effcc>] SYSC_finit_module+0xbc/0xf0
 [<ffffffff810f001e>] SyS_finit_module+0xe/0x10
 [<ffffffff8152fe40>] entry_SYSCALL_64_fastpath+0x23/0xc1

WARNING: CPU: 2 PID: 12885 at scst/src/scst_copy_mgr.c:2826 scst_cm_check_access_acg+0x180/0x1e0 [scst]
Call Trace:
 [<ffffffff812a4d23>] dump_stack+0x86/0xc3
 [<ffffffff8106412b>] __warn+0xcb/0xf0
 [<ffffffff8106421d>] warn_slowpath_null+0x1d/0x20
 [<ffffffffa0502b00>] scst_cm_check_access_acg+0x180/0x1e0 [scst]
 [<ffffffffa0502cba>] scst_cm_check_access+0x15a/0x280 [scst]
 [<ffffffffa05030cb>] scst_cm_parse_id_tgt_descr+0x2eb/0x590 [scst]
 [<ffffffffa050a1d8>] scst_cm_parse_descriptors+0x308/0xa60 [scst]
 [<ffffffffa04d4440>] scst_parse_descriptors+0x60/0x130 [scst]
 [<ffffffffa04a3c05>] scst_tgt_pre_exec+0x375/0x4a0 [scst]
 [<ffffffffa04ac2a9>] scst_process_active_cmd+0x439/0x740 [scst]
 [<ffffffffa04adbb2>] scst_cmd_thread+0x202/0x6a0 [scst]
 [<ffffffff810883cf>] kthread+0xef/0x110
 [<ffffffff8153008f>] ret_from_fork+0x1f/0x40


git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@7097 d57e44dd-8a1f-0410-8b47-8ef2f437770f
 2017-02-24 07:07:41 +00:00
Bart Van Assche
bc85808150 Merge branch 'svn-trunk' 2017-02-23 18:38:45 -08:00
Bart Van Assche
ab0e7cc0be scst_copy_mgr: Avoid that LUN removal triggers a BUG() 
Reported-by: Jinpu Wang <jinpu.wang@profitbricks.com>


git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@7096 d57e44dd-8a1f-0410-8b47-8ef2f437770f
 2017-02-24 02:08:17 +00:00
Bart Van Assche
585c5d2fe0 scst_copy_mgr: Convert comments about locking into runtime checks 
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@7095 d57e44dd-8a1f-0410-8b47-8ef2f437770f
 2017-02-24 02:07:29 +00:00
Bart Van Assche
621cd3adf7 Merge branch 'svn-trunk' 2017-02-07 18:28:44 -08:00
Bart Van Assche
2a074789ac scstadmin: Revert r7014 
The immediateExit() call introduced in r7014 prevents to assign multiple
values to attributes that support multiple values, e.g. 'allowed_portal'.
Hence revert r7014.


git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@7092 d57e44dd-8a1f-0410-8b47-8ef2f437770f
 2017-02-05 06:52:06 +00:00
Bart Van Assche
bddec074a2 scst_sgv: Convert to kernel v4.10 2017-02-02 13:44:45 -08:00
Bart Van Assche
361bba626f ib_srpt: Fix ib_dma_map_ops test 2017-02-02 13:44:44 -08:00
Bart Van Assche
ce4edcf7b3 ib_srpt: Fix a copy/paste bug 2017-02-02 13:44:44 -08:00
Bart Van Assche
e45cbab700 scst: Port to Linux kernel v4.10 2017-02-02 13:44:43 -08:00
Vladislav Bolkhovitin
5930008dba Add scstadmin in the dist archive 
Reported by David Chen <david.chen@osnexus.com>



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@7091 d57e44dd-8a1f-0410-8b47-8ef2f437770f
 2017-01-24 00:29:41 +00:00
Bart Van Assche
cf185c85c0 Merge branch 'svn-trunk' 2017-01-10 17:52:45 -08:00
Bart Van Assche
4538bc1c81 scst: Fix a memory leak 
Avoid that the memory allocated for struct scst_acn and its name is
leaked by scst_del_acg().


git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@7090 d57e44dd-8a1f-0410-8b47-8ef2f437770f
 2017-01-11 01:16:07 +00:00
Bart Van Assche
473db9c997 Merge branch 'svn-trunk' 2017-01-08 13:46:29 -08:00
Bart Van Assche
b51c624855 ib_srpt: Only report that SRQ creation failed if it was attempted to create an SRQ 
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@7089 d57e44dd-8a1f-0410-8b47-8ef2f437770f
 2017-01-08 21:46:20 +00:00
Bart Van Assche
3abcc849df ib_srpt: Destroy SRQ before freeing the buffers posted on the SRQ 
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@7088 d57e44dd-8a1f-0410-8b47-8ef2f437770f
 2017-01-08 21:45:39 +00:00
Bart Van Assche
db686530c9 ib_srpt: Rename a symbolic constant 
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@7087 d57e44dd-8a1f-0410-8b47-8ef2f437770f
 2017-01-08 21:10:54 +00:00
Bart Van Assche
c9b2dfa070 ib_srpt: Remove a local variable 
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@7086 d57e44dd-8a1f-0410-8b47-8ef2f437770f
 2017-01-08 21:10:32 +00:00