)]}' { "log": [ { "commit": "f475f6775465283494346663f201ad04810d2e8a", "tree": "b5c054afa5509ee6957fb6a93d09999b9db975ab", "parents": [ "ec06aedd44541129840ed52e6165afa3796a27bf" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Fri Nov 06 14:18:49 2009 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Nov 06 22:06:14 2009 +0000" }, "message": "cifs: don\u0027t use CIFSGetSrvInodeNumber in is_path_accessible\n\nBecause it\u0027s lighter weight, CIFS tries to use CIFSGetSrvInodeNumber to\nverify the accessibility of the root inode and then falls back to doing a\nfull QPathInfo if that fails with -EOPNOTSUPP. I have at least a report\nof a server that returns NT_STATUS_INTERNAL_ERROR rather than something\nthat translates to EOPNOTSUPP.\n\nRather than trying to be clever with that call, just have\nis_path_accessible do a normal QPathInfo. That call is widely\nsupported and it shouldn\u0027t increase the overhead significantly.\n\nCc: Stable \u003cstable@kernel.org\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "ec06aedd44541129840ed52e6165afa3796a27bf", "tree": "3099a49ef7aac9ea14a921559ba067b230e33948", "parents": [ "7c9abfb884b8737f0afdc8a88bcea77526f0da87" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Fri Nov 06 14:18:29 2009 -0500" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Nov 06 22:04:37 2009 +0000" }, "message": "cifs: clean up handling when server doesn\u0027t consistently support inode numbers\n\nIt\u0027s possible that a server will return a valid FileID when we query the\nFILE_INTERNAL_INFO for the root inode, but then zeroed out inode numbers\nwhen we do a FindFile with an infolevel of\nSMB_FIND_FILE_ID_FULL_DIR_INFO.\n\nIn this situation turn off querying for server inode numbers, generate a\nwarning for the user and just generate an inode number using iunique.\nOnce we generate any inode number with iunique we can no longer use any\nserver inode numbers or we risk collisions, so ensure that we don\u0027t do\nthat in cifs_get_inode_info either.\n\nCc: Stable \u003cstable@kernel.org\u003e\nReported-by: Timothy Normand Miller \u003ctheosib@gmail.com\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "8347a5cdd1422eea0470ed586274c7f29e274b47", "tree": "cc83dbc0307c9d2393078e27d01e9f451eccd578", "parents": [ "0d9df2515dbceb67d343c0f10fd3ff218380d524" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Oct 06 18:31:29 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Oct 06 18:31:29 2009 +0000" }, "message": "[CIFS] Fixing to avoid invalid kfree() in cifs_get_tcp_session()\n\ntrivial bug in fs/cifs/connect.c .\nThe bug is caused by fail of extract_hostname()\nwhen mounting cifs file system.\n\nThis is the situation when I noticed this bug.\n\n% sudo mount -t cifs //192.168.10.208 mountpoint -o options...\n\nThen my kernel says,\n\n[ 1461.807776] ------------[ cut here ]------------\n[ 1461.807781] kernel BUG at mm/slab.c:521!\n[ 1461.807784] invalid opcode: 0000 [#2] PREEMPT SMP\n[ 1461.807790] last sysfs file:\n/sys/devices/pci0000:00/0000:00:1e.0/0000:09:02.0/resource\n[ 1461.807793] CPU 0\n[ 1461.807796] Modules linked in: nls_iso8859_1 usbhid sbp2 uhci_hcd\nehci_hcd i2c_i801 ohci1394 ieee1394 psmouse serio_raw pcspkr sky2 usbcore\nevdev\n[ 1461.807816] Pid: 3446, comm: mount Tainted: G D 2.6.32-rc2-vanilla\n[ 1461.807820] RIP: 0010:[\u003cffffffff810b888e\u003e] [\u003cffffffff810b888e\u003e]\nkfree+0x63/0x156\n[ 1461.807829] RSP: 0018:ffff8800b4f7fbb8 EFLAGS: 00010046\n[ 1461.807832] RAX: ffffea00033fff98 RBX: ffff8800afbae7e2 RCX:\n0000000000000000\n[ 1461.807836] RDX: ffffea0000000000 RSI: 000000000000005c RDI:\nffffffffffffffea\n[ 1461.807839] RBP: ffff8800b4f7fbf8 R08: 0000000000000001 R09:\n0000000000000000\n[ 1461.807842] R10: 0000000000000000 R11: ffff8800b4f7fbf8 R12:\n00000000ffffffea\n[ 1461.807845] R13: ffff8800afb23000 R14: ffff8800b4f87bc0 R15:\nffffffffffffffea\n[ 1461.807849] FS: 00007f52b6f187c0(0000) GS:ffff880007600000(0000)\nknlGS:0000000000000000\n[ 1461.807852] CS: 0010 DS: 0000 ES: 0000 CR0: 000000008005003b\n[ 1461.807855] CR2: 0000000000613000 CR3: 00000000af8f9000 CR4:\n00000000000006f0\n[ 1461.807858] DR0: 0000000000000000 DR1: 0000000000000000 DR2:\n0000000000000000\n[ 1461.807861] DR3: 0000000000000000 DR6: 00000000ffff0ff0 DR7:\n0000000000000400\n[ 1461.807865] Process mount (pid: 3446, threadinfo ffff8800b4f7e000, task\nffff8800950e4380)\n[ 1461.807867] Stack:\n[ 1461.807869] 0000000000000202 0000000000000282 ffff8800b4f7fbf8\nffff8800afbae7e2\n[ 1461.807876] \u003c0\u003e 00000000ffffffea ffff8800afb23000 ffff8800b4f87bc0\nffff8800b4f7fc28\n[ 1461.807884] \u003c0\u003e ffff8800b4f7fcd8 ffffffff81159f6d ffffffff81147bc2\nffffffff816bfb48\n[ 1461.807892] Call Trace:\n[ 1461.807899] [\u003cffffffff81159f6d\u003e] cifs_get_tcp_session+0x440/0x44b\n[ 1461.807904] [\u003cffffffff81147bc2\u003e] ? find_nls+0x1c/0xe9\n[ 1461.807909] [\u003cffffffff8115b889\u003e] cifs_mount+0x16bc/0x2167\n[ 1461.807917] [\u003cffffffff814455bd\u003e] ? _spin_unlock+0x30/0x4b\n[ 1461.807923] [\u003cffffffff81150da9\u003e] cifs_get_sb+0xa5/0x1a8\n[ 1461.807928] [\u003cffffffff810c1b94\u003e] vfs_kern_mount+0x56/0xc9\n[ 1461.807933] [\u003cffffffff810c1c64\u003e] do_kern_mount+0x47/0xe7\n[ 1461.807938] [\u003cffffffff810d8632\u003e] do_mount+0x712/0x775\n[ 1461.807943] [\u003cffffffff810d671f\u003e] ? copy_mount_options+0xcf/0x132\n[ 1461.807948] [\u003cffffffff810d8714\u003e] sys_mount+0x7f/0xbf\n[ 1461.807953] [\u003cffffffff8144509a\u003e] ? lockdep_sys_exit_thunk+0x35/0x67\n[ 1461.807960] [\u003cffffffff81011cc2\u003e] system_call_fastpath+0x16/0x1b\n[ 1461.807963] Code: 00 00 00 00 ea ff ff 48 c1 e8 0c 48 6b c0 68 48 01 d0\n66 83 38 00 79 04 48 8b 40 10 66 83 38 00 79 04 48 8b 40 10 80 38 00 78 04\n\u003c0f\u003e 0b eb fe 4c 8b 70 58 4c 89 ff 41 8b 76 4c e8 b8 49 fb ff e8\n[ 1461.808022] RIP [\u003cffffffff810b888e\u003e] kfree+0x63/0x156\n[ 1461.808027] RSP \u003cffff8800b4f7fbb8\u003e\n[ 1461.808031] ---[ end trace ffe26fcdc72c0ce4 ]---\n\nThe reason of this bug is that the error handling code of\ncifs_get_tcp_session()\ncalls kfree() when corresponding kmalloc() failed.\n(The kmalloc() is called by extract_hostname().)\n\nSigned-off-by: Hitoshi Mitake \u003cmitake@dcl.info.waseda.ac.jp\u003e\nCC: Stable \u003cstable@kernel.org\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "07e2e6ba2761291aa182993ebba1f490b5005dcc", "tree": "4179e6189472f3a2434bb5cae01ec8f694881955", "parents": [ "d8f654ef6a55495e548427b997a388e0d5a1ffb4", "3321b791b2e8897323f8c044a0c77ff25781381c" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Sat Sep 26 10:10:35 2009 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Sat Sep 26 10:10:35 2009 -0700" }, "message": "Merge git://git.kernel.org/pub/scm/linux/kernel/git/sfrench/cifs-2.6\n\n* git://git.kernel.org/pub/scm/linux/kernel/git/sfrench/cifs-2.6:\n cifs: fix locking and list handling code in cifs_open and its helper\n [CIFS] Remove build warning\n cifs: fix problems with last two commits\n [CIFS] Fix build break when keys support turned off\n cifs: eliminate cifs_init_private\n cifs: convert oplock breaks to use slow_work facility (try #4)\n cifs: have cifsFileInfo hold an extra inode reference\n cifs: take read lock on GlobalSMBSes_lock in is_valid_oplock_break\n cifs: remove cifsInodeInfo.oplockPending flag\n cifs: fix oplock request handling in posix codepath\n [CIFS] Re-enable Lanman security\n" }, { "commit": "3321b791b2e8897323f8c044a0c77ff25781381c", "tree": "cf349723b7b4f48151d10eeaeff322c05fd9de3b", "parents": [ "15dd478107fb110689ef09d276d84051b31b7e5c" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Fri Sep 25 09:53:37 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Sep 25 17:59:31 2009 +0000" }, "message": "cifs: fix locking and list handling code in cifs_open and its helper\n\nThe patch to remove cifs_init_private introduced a locking imbalance. It\ndidn\u0027t remove the leftover list addition code and the unlocking in that\nfunction. cifs_new_fileinfo does the list addition now, so there should\nbe no need to do it outside of that function.\n\npCifsInode will never be NULL, so we don\u0027t need to check for that. This\npatch also gets rid of the ugly locking and unlocking across function\ncalls.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nAcked-by: Steve French \u003csfrench@us.ibm.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "15dd478107fb110689ef09d276d84051b31b7e5c", "tree": "09a1480b4c4e1425d1f40853d9a31c93bc9faf9f", "parents": [ "5d2c0e225982c667167264a62252308f78727f44" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Sep 25 02:24:45 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Sep 25 02:24:45 2009 +0000" }, "message": "[CIFS] Remove build warning\n\nAcked-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "5d2c0e225982c667167264a62252308f78727f44", "tree": "9a9251b9e143be20c7d707e0c9065402ecaf1e82", "parents": [ "0f59e61c1f034b8088086bc85dc777af2ad3e836" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Sep 24 20:35:33 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Sep 25 02:12:33 2009 +0000" }, "message": "cifs: fix problems with last two commits\n\nFix problems with commits:\n\n086f68bd97126618ecb2dcff5f766f3a21722df7\n3bc303c254335dbd7c7012cc1760b12f1d5514d3\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "0f59e61c1f034b8088086bc85dc777af2ad3e836", "tree": "a126a404e29aea76619129c3af5840f9582fec43", "parents": [ "086f68bd97126618ecb2dcff5f766f3a21722df7" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Sep 25 00:33:37 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Sep 25 00:33:37 2009 +0000" }, "message": "[CIFS] Fix build break when keys support turned off\n\nAcked-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "086f68bd97126618ecb2dcff5f766f3a21722df7", "tree": "25f5785dc4ce4ffdc8c852b6c04e52324dd334b0", "parents": [ "3bc303c254335dbd7c7012cc1760b12f1d5514d3" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Mon Sep 21 14:08:18 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Sep 24 19:35:18 2009 +0000" }, "message": "cifs: eliminate cifs_init_private\n\n...it does the same thing as cifs_fill_fileinfo, but doesn\u0027t handle the\nflist ordering correctly. Also rename cifs_fill_fileinfo to a more\ndescriptive name and have it take an open flags arg instead of just a\nwrite_only flag. That makes the logic in the callers a little simpler.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "3bc303c254335dbd7c7012cc1760b12f1d5514d3", "tree": "7da17fbfd697216d9ed0ccd64ea9c03aaf3d52c1", "parents": [ "48541bd3dd4739b4d574b44ea47660c88d833677" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Mon Sep 21 06:47:50 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Sep 24 18:33:18 2009 +0000" }, "message": "cifs: convert oplock breaks to use slow_work facility (try #4)\n\nThis is the fourth respin of the patch to convert oplock breaks to\nuse the slow_work facility.\n\nA customer of ours was testing a backport of one of the earlier\npatchsets, and hit a \"Busy inodes after umount...\" problem. An oplock\nbreak job had raced with a umount, and the superblock got torn down and\nits memory reused. When the oplock break job tried to dereference the\ninode-\u003ei_sb, the kernel oopsed.\n\nThis patchset has the oplock break job hold an inode and vfsmount\nreference until the oplock break completes. With this, there should be\nno need to take a tcon reference (the vfsmount implicitly holds one\nalready).\n\nCurrently, when an oplock break comes in there\u0027s a chance that the\noplock break job won\u0027t occur if the allocation of the oplock_q_entry\nfails. There are also some rather nasty races in the allocation and\nhandling these structs.\n\nRather than allocating oplock queue entries when an oplock break comes\nin, add a few extra fields to the cifsFileInfo struct. Get rid of the\ndedicated cifs_oplock_thread as well and queue the oplock break job to\nthe slow_work thread pool.\n\nThis approach also has the advantage that the oplock break jobs can\npotentially run in parallel rather than be serialized like they are\ntoday.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "c08d3b0e33edce28e9cfa7b64f7fe5bdeeb29248", "tree": "6c161cc57a2873e6f91335113192a99531901cd8", "parents": [ "25d9e2d15286281ec834b829a4aaf8969011f1cd" ], "author": { "name": "npiggin@suse.de", "email": "npiggin@suse.de", "time": "Fri Aug 21 02:35:06 2009 +1000" }, "committer": { "name": "al", "email": "al@dizzy.pdmi.ras.ru", "time": "Thu Sep 24 08:41:47 2009 -0400" }, "message": "truncate: use new helpers\n\nUpdate some fs code to make use of new helper functions introduced\nin the previous patch. Should be no significant change in behaviour\n(except CIFS now calls send_sig under i_lock, via inode_newsize_ok).\n\nReviewed-by: Christoph Hellwig \u003chch@lst.de\u003e\nAcked-by: Miklos Szeredi \u003cmiklos@szeredi.hu\u003e\nCc: linux-nfs@vger.kernel.org\nCc: Trond.Myklebust@netapp.com\nCc: linux-cifs-client@lists.samba.org\nCc: sfrench@samba.org\nSigned-off-by: Nick Piggin \u003cnpiggin@suse.de\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "6d729e44a55547c009d7a87ea66bff21a8e0afea", "tree": "c27e69babb9b397ad56cb20a935b75c35ceb364f", "parents": [ "4504230a71566785a05d3e6b53fa1ee071b864eb" ], "author": { "name": "Thomas Gleixner", "email": "tglx@linutronix.de", "time": "Sun Aug 16 21:05:08 2009 +0000" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Thu Sep 24 07:47:42 2009 -0400" }, "message": "fs: Make unload_nls() NULL pointer safe\n\nMost call sites of unload_nls() do:\n\tif (nls)\n\t\tunload_nls(nls);\n\nCheck the pointer inside unload_nls() like we do in kfree() and\nsimplify the call sites.\n\nSigned-off-by: Thomas Gleixner \u003ctglx@linutronix.de\u003e\nCc: Steve French \u003csfrench@us.ibm.com\u003e\nCc: OGAWA Hirofumi \u003chirofumi@mail.parknet.co.jp\u003e\nCc: Roman Zippel \u003czippel@linux-m68k.org\u003e\nCc: Dave Kleikamp \u003cshaggy@linux.vnet.ibm.com\u003e\nCc: Petr Vandrovec \u003cvandrove@vc.cvut.cz\u003e\nCc: Anton Altaparmakov \u003caia21@cantab.net\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "342ff1a1b558ebbdb8cbd55ab6a63eca8b2473ca", "tree": "1f967f283dade6e03897169bb29513354f49f910", "parents": [ "50223e486cabdcf7e540e519da1f26bab3084e5d", "24ed7a97464db44592495f98cff8bcee02f92bc2" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Sep 22 07:51:45 2009 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Sep 22 07:51:45 2009 -0700" }, "message": "Merge branch \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/jikos/trivial\n\n* \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/jikos/trivial: (34 commits)\n trivial: fix typo in aic7xxx comment\n trivial: fix comment typo in drivers/ata/pata_hpt37x.c\n trivial: typo in kernel-parameters.txt\n trivial: fix typo in tracing documentation\n trivial: add __init/__exit macros in drivers/gpio/bt8xxgpio.c\n trivial: add __init macro/ fix of __exit macro location in ipmi_poweroff.c\n trivial: remove unnecessary semicolons\n trivial: Fix duplicated word \"options\" in comment\n trivial: kbuild: remove extraneous blank line after declaration of usage()\n trivial: improve help text for mm debug config options\n trivial: doc: hpfall: accept disk device to unload as argument\n trivial: doc: hpfall: reduce risk that hpfall can do harm\n trivial: SubmittingPatches: Fix reference to renumbered step\n trivial: fix typos \"man[ae]g?ment\" -\u003e \"management\"\n trivial: media/video/cx88: add __init/__exit macros to cx88 drivers\n trivial: fix typo in CONFIG_DEBUG_FS in gcov doc\n trivial: fix missing printk space in amd_k7_smp_check\n trivial: fix typo s/ketymap/keymap/ in comment\n trivial: fix typo \"to to\" in multiple files\n trivial: fix typos in comments s/DGBU/DBGU/\n ...\n" }, { "commit": "6e1d5dcc2bbbe71dbf010c747e15739bef6b7218", "tree": "2edb0f6cc65acbae95e42df1bc763ec048e6c2e0", "parents": [ "7f09410bbc4306f592cfb43812389ea1c7905a20" ], "author": { "name": "Alexey Dobriyan", "email": "adobriyan@gmail.com", "time": "Mon Sep 21 17:01:11 2009 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Sep 22 07:17:24 2009 -0700" }, "message": "const: mark remaining inode_operations as const\n\nSigned-off-by: Alexey Dobriyan \u003cadobriyan@gmail.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "0d54b217a247f39605361f867fefbb9e099a5432", "tree": "5af6a2cc4cbc6c327e735ac5edb642d47a1f0013", "parents": [ "61e225dc341107be304fd1088146c2a5e88ff9e0" ], "author": { "name": "Alexey Dobriyan", "email": "adobriyan@gmail.com", "time": "Mon Sep 21 17:01:09 2009 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Sep 22 07:17:24 2009 -0700" }, "message": "const: make struct super_block::s_qcop const\n\nSigned-off-by: Alexey Dobriyan \u003cadobriyan@gmail.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "a419aef8b858a2bdb98df60336063d28df4b272f", "tree": "1736f6650ec0bfc01074c489fc47396114099c5e", "parents": [ "2944fcbe03d65a704f07e43efe14adb0d226fd09" ], "author": { "name": "Joe Perches", "email": "joe@perches.com", "time": "Tue Aug 18 11:18:35 2009 -0700" }, "committer": { "name": "Jiri Kosina", "email": "jkosina@suse.cz", "time": "Mon Sep 21 15:14:58 2009 +0200" }, "message": "trivial: remove unnecessary semicolons\n\nSigned-off-by: Joe Perches \u003cjoe@perches.com\u003e\nSigned-off-by: Jiri Kosina \u003cjkosina@suse.cz\u003e\n" }, { "commit": "48541bd3dd4739b4d574b44ea47660c88d833677", "tree": "5ee6d16b924b8c171e66088c5bb3b1d1720cd29f", "parents": [ "058daf4f6795fd23fcca41fe53151cdcbbd263ec" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Sat Sep 12 11:54:29 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Sep 15 19:45:25 2009 +0000" }, "message": "cifs: have cifsFileInfo hold an extra inode reference\n\nIt\u0027s possible that this struct will outlive the filp to which it is\nattached. If it does and it needs to do some work on the inode, then\nit\u0027ll need a reference.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "058daf4f6795fd23fcca41fe53151cdcbbd263ec", "tree": "20efa19e535a71cded4ae965a26fe7bcd227525a", "parents": [ "495e993745a722cfa54d76b97d18e8f30c3e6cea" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Sat Sep 12 11:54:29 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Sep 15 19:45:10 2009 +0000" }, "message": "cifs: take read lock on GlobalSMBSes_lock in is_valid_oplock_break\n\n...rather than a write lock. It doesn\u0027t change the list so a read lock\nshould be sufficient.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "495e993745a722cfa54d76b97d18e8f30c3e6cea", "tree": "87f7f3cee5d6dec98debc8957628548f2b774ed2", "parents": [ "590a3fe0e18473a55b0ff48548ba87a2010d7730" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Sat Sep 12 11:54:29 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Sep 15 19:45:06 2009 +0000" }, "message": "cifs: remove cifsInodeInfo.oplockPending flag\n\nIt\u0027s set on oplock break but nothing ever looks at it.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "590a3fe0e18473a55b0ff48548ba87a2010d7730", "tree": "0e5fdfa70ee40a10a5b0df3b85cef3baa0d1b2c6", "parents": [ "20d1752f3d6bd32beb90949559e0d14a0b234445" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Sat Sep 12 11:54:28 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Sep 15 19:45:03 2009 +0000" }, "message": "cifs: fix oplock request handling in posix codepath\n\ncifs_posix_open takes a \"poplock\" argument that\u0027s intended to be used in\nthe actual posix open call to set the \"Flags\" field. It ignores this\nvalue however and declares an \"oplock\" parameter on the stack that it\npasses uninitialized to the CIFSPOSIXOpen function. Not only does this\nmean that the oplock request flags are bogus, but the result that\u0027s\nexpected to be in that variable is unchanged.\n\nFix this, and also clean up the type of the oplock parameter used. Since\nit\u0027s expected to be __u32, we should use that everywhere and not\nimplicitly cast it from a signed type.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "20d1752f3d6bd32beb90949559e0d14a0b234445", "tree": "4c3c4370b8ad18e07339b4f9e53d35c6df049512", "parents": [ "4142e0d1def2c0176c27fd2e810243045a62eb6d" ], "author": { "name": "Chuck Ebbert", "email": "cebbert@redhat.com", "time": "Tue Sep 15 01:53:21 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Sep 15 19:38:05 2009 +0000" }, "message": "[CIFS] Re-enable Lanman security\n\ncommit ac68392460ffefed13020967bae04edc4d3add06 (\"[CIFS] Allow raw\nntlmssp code to be enabled with sec\u003dntlmssp\") added a new bit to the\nallowed security flags mask but seems to have inadvertently removed\nLanman security from the allowed flags. Add it back.\n\nCC: Stable \u003cstable@kernel.org\u003e\nSigned-off-by: Chuck Ebbert \u003ccebbert@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "9162ab2000e08be076883b5a295a771223264ce8", "tree": "95a51f8e1fe17a3916ed77554ac6de7e0c82b556", "parents": [ "6ab409b53dcaf28f83d518a6702f904b7cee3f41" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Sep 03 12:07:17 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Sep 03 15:30:48 2009 +0000" }, "message": "cifs: consolidate reconnect logic in smb_init routines\n\nThere\u0027s a large cut and paste chunk of code in smb_init and\nsmall_smb_init to handle reconnects. Break it out into a separate\nfunction, clean it up and have both routines call it.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "6ab409b53dcaf28f83d518a6702f904b7cee3f41", "tree": "050bfb690ac9df049343034681478a5bb174a823", "parents": [ "1b49c5566136455764a8d17ead25784f534c202d" ], "author": { "name": "Dave Kleikamp", "email": "shaggy@linux.vnet.ibm.com", "time": "Mon Aug 31 11:07:12 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Sep 01 22:35:01 2009 +0000" }, "message": "cifs: Replace wrtPending with a real reference count\n\nCurrently, cifs_close() tries to wait until all I/O is complete and then\nfrees the file private data. If I/O does not completely in a reasonable\namount of time it frees the structure anyway, leaving a potential use-\nafter-free situation.\n\nThis patch changes the wrtPending counter to a complete reference count and\nlets the last user free the structure.\n\nSigned-off-by: Dave Kleikamp \u003cshaggy@linux.vnet.ibm.com\u003e\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nTested-by: Shirish Pargaonkar \u003cshirishp@us.ibm.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "1b49c5566136455764a8d17ead25784f534c202d", "tree": "6946842743140355a6f6b1fb1f27fbfa59132337", "parents": [ "8e047d09ee1143058b71f40c5f4d028dde52d883" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Fri Aug 28 10:11:11 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Sep 01 22:25:29 2009 +0000" }, "message": "cifs: protect GlobalOplock_Q with its own spinlock\n\nRight now, the GlobalOplock_Q is protected by the GlobalMid_Lock. That\nlock is also used for completely unrelated purposes (mostly for managing\nthe global mid queue). Give the list its own dedicated spinlock\n(cifs_oplock_lock) and rename the list to cifs_oplock_list to\neliminate the camel-case.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "8e047d09ee1143058b71f40c5f4d028dde52d883", "tree": "05db83ba0f9c5a4af824d60c3b15f4c0204b615b", "parents": [ "8c58b54574d18b3782b2a261b9dae3c4e90b6b04" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Fri Aug 28 10:11:10 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Sep 01 22:25:19 2009 +0000" }, "message": "cifs: use tcon pointer in cifs_show_options\n\nMinor nit: we already have a tcon pointer so we don\u0027t need to\ndereference cifs_sb again.\n\nAlso initialize the vars in the declaration.\n\nReported-by: Peter Staubach \u003cstaubach@redhat.com\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "8c58b54574d18b3782b2a261b9dae3c4e90b6b04", "tree": "76e32545f819dce52b1cb0bdc6236dd539c271bc", "parents": [ "ca43e3beee38623a3f80691b28623d6ecf214742" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Mon Aug 17 16:26:49 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Sep 01 22:24:10 2009 +0000" }, "message": "cifs: send IPv6 addr in upcall with colon delimiters\n\nMake it easier on the upcall program by adding \u0027:\u0027 delimiters between\neach group of hex digits.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "ca43e3beee38623a3f80691b28623d6ecf214742", "tree": "779d99d0f4a29d3fc6541f48423da9852eb6bbbe", "parents": [ "bdb97adcdf0993adbd2eef44b4533620d43792de" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Sep 01 17:20:50 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Sep 01 17:20:50 2009 +0000" }, "message": "[CIFS] Fix checkpatch warnings\n\nAlso update version number to 1.61\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "bdb97adcdf0993adbd2eef44b4533620d43792de", "tree": "5d2a120233c93d6a372d26d5ce3fc6cb6525fa6e", "parents": [ "1b3859bc9e20d764316346665fc93ecea2d2b176" ], "author": { "name": "Suresh Jayaraman", "email": "sjayaraman@suse.de", "time": "Thu Aug 20 13:03:34 2009 +0530" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Sep 01 17:08:48 2009 +0000" }, "message": "PATCH] cifs: fix broken mounts when a SSH tunnel is used (try #4)\n\nOne more try..\n\nIt seems there is a regression that got introduced while Jeff fixed\nall the mount/umount races. While attempting to find whether a tcp\nsession is already existing, we were not checking whether the \"port\"\nused are the same. When a second mount is attempted with a different\n\"port\u003d\" option, it is being ignored. Because of this the cifs mounts\nthat uses a SSH tunnel appears to be broken.\n\nSteps to reproduce:\n\n1. create 2 shares\n# SSH Tunnel a SMB session\n2. ssh -f -L 6111:127.0.0.1:445 root@localhost \"sleep 86400\"\n3. ssh -f -L 6222:127.0.0.1:445 root@localhost \"sleep 86400\"\n4. tcpdump -i lo 6111 \u0026\n5. mkdir -p /mnt/mnt1\n6. mkdir -p /mnt/mnt2\n7. mount.cifs //localhost/a /mnt/mnt1 -o username\u003dguest,ip\u003d127.0.0.1,port\u003d6111\n#(shows tcpdump activity on port 6111)\n8. mount.cifs //localhost/b /mnt/mnt2 -o username\u003dguest,ip\u003d127.0.0.1,port\u003d6222\n#(shows tcpdump activity only on port 6111 and not on 6222\n\nFix by adding a check to compare the port _only_ if the user tries to\noverride the tcp port with \"port\u003d\" option, before deciding that an\nexisting tcp session is found. Also, clean up a bit by replacing\nif-else if by a switch statment while at it as suggested by Jeff.\n\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Shirish Pargaonkar \u003cshirishp@us.ibm.com\u003e\nSigned-off-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "1b3859bc9e20d764316346665fc93ecea2d2b176", "tree": "9a643e0bf74d9b8339ae4bc45df1988ed306dc4d", "parents": [ "2920ee2b47fc8e6aebe1d1956b2725f48fa93cc5" ], "author": { "name": "Alexander Strakh", "email": "strakh@ispras.ru", "time": "Tue Sep 01 17:02:24 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Sep 01 17:02:24 2009 +0000" }, "message": "[CIFS] Memory leak in ntlmv2 hash calculation\n\n in function calc_ntlmv2_hash memory is not released.\n1. If in the line 333 we successfully allocate memory and assign it to\npctxt variable:\n pctxt \u003d kmalloc(sizeof(struct HMACMD5Context), GFP_KERNEL);\nthen we go to line 376 and exit wihout releasing memory pointed to by pctxt\nvariable.\n\nAdd a memory releasing for pctxt variable before exit from function\ncalc_ntlmv2_hash.\n\nSigned-off-by: Alexander Strakh \u003cstrakh@ispras.ru\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "2920ee2b47fc8e6aebe1d1956b2725f48fa93cc5", "tree": "d6c65477946df8f136800e813da52df55e65919c", "parents": [ "7b2aa037e878c939676675969983284a02958ae3" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Aug 31 15:27:26 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Aug 31 15:27:26 2009 +0000" }, "message": "[CIFS] potential NULL dereference in parse_DFS_referrals()\n\nmemory allocation may fail, prevent a NULL dereference\n\nPointed out by Roel Kluin\n\nCC: Roel Kluin \u003croel.kluin@gmail.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "d098564f3b2b5d555e51bca765a6a9e0dda8f2cd", "tree": "4408cca53ee3a1c024514d1dd06cedd58bc5af48", "parents": [ "24e2fb615fd6b624c320cec9ea9d91a75dad902e" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Aug 04 03:53:28 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Aug 04 03:53:28 2009 +0000" }, "message": "[CIFS] Update readme to reflect forceuid mount parms\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "24e2fb615fd6b624c320cec9ea9d91a75dad902e", "tree": "f6849650d675636490a14659228ce9a18004bc77", "parents": [ "4486d6ede16b362f89b29845af6fe1a26ae78a54" ], "author": { "name": "Roel Kluin", "email": "roel.kluin@gmail.com", "time": "Sun Aug 02 13:00:18 2009 +0200" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Aug 03 21:01:32 2009 +0000" }, "message": "cifs: Read buffer overflow\n\nCheck whether index is within bounds before testing the element.\n\nAcked-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Roel Kluin \u003croel.kluin@gmail.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "4486d6ede16b362f89b29845af6fe1a26ae78a54", "tree": "b96bb887453baccdcfe1a0afee0f6cc9e7aaae4a", "parents": [ "9b9d6b2434fe942895c341b9a982f158529788ec" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Mon Aug 03 12:45:10 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Aug 03 21:01:17 2009 +0000" }, "message": "cifs: show noforceuid/noforcegid mount options (try #2)\n\nSince forceuid is the default, we now need to show when it\u0027s disabled.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "9b9d6b2434fe942895c341b9a982f158529788ec", "tree": "402b7f233fae53f45d5ad3a445360200b6fa2cc7", "parents": [ "5bd9052d79daa4c8beb45436c408b6de672adb82" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Fri Jul 31 06:56:09 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sun Aug 02 03:47:25 2009 +0000" }, "message": "cifs: reinstate original behavior when uid\u003d/gid\u003d options are specified\n\nThis patch fixes the regression reported here:\n\nhttp://bugzilla.kernel.org/show_bug.cgi?id\u003d13861\n\ncommit 4ae1507f6d266d0cc3dd36e474d83aad70fec9e4 changed the default\nbehavior when the uid\u003d or gid\u003d option was specified for a mount. The\nexisting behavior was to always clobber the ownership information\nprovided by the server when these options were specified. The above\ncommit changed this behavior so that these options simply provided\ndefaults when the server did not provide this information (unless\n\"forceuid\" or \"forcegid\" were specified)\n\nThis patch reverts this change so that the default behavior is restored.\nIt also adds \"noforceuid\" and \"noforcegid\" options to make it so that\nownership information from the server is preserved, even when the mount\nhas uid\u003d or gid\u003d options specified.\n\nIt also adds a couple of printk notices that pop up when forceuid or\nforcegid options are specified without a uid\u003d or gid\u003d option.\n\nReported-by: Tom Chiverton \u003cbugzilla.kernel.org@falkensweb.com\u003e\nReviewed-by: Shirish Pargaonkar \u003cshirishp@us.ibm.com\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "5bd9052d79daa4c8beb45436c408b6de672adb82", "tree": "de976b18a0bbea6c48801a88aec56d618c4f78dc", "parents": [ "7b91e2661addd8e2419cb45f6a322aa5dab9bcee" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Jul 30 02:26:14 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Jul 30 02:26:14 2009 +0000" }, "message": "[CIFS] Updates fs/cifs/CHANGES\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "7b91e2661addd8e2419cb45f6a322aa5dab9bcee", "tree": "cf17b0ca2bffa52c7e450cd75d5277c35cc9eda4", "parents": [ "fc013a58859b7cf85e53a05804a74952fe0a4117" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Jul 23 15:22:30 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Jul 28 00:51:59 2009 +0000" }, "message": "cifs: fix error handling in mount-time DFS referral chasing code\n\nIf the referral is malformed or the hostname can\u0027t be resolved, then\nthe current code generates an oops. Fix it to handle these errors\ngracefully.\n\nReported-by: Sandro Mathys \u003csm@sandro-mathys.ch\u003e\nAcked-by: Igor Mammedov \u003cniallain@gmail.com\u003e\nCC: Stable \u003cstable@kernel.org\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "f1230c97978f52268d8c66e6f88e54c3d2092a75", "tree": "3ec132331d1eb944b464be772469b464a8c6a26d", "parents": [ "03aa3a49ad3592a9e4e1ab19c6da3e852288caf1" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Jul 22 23:13:01 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Jul 22 23:13:01 2009 +0000" }, "message": "[CIFS] fix sparse warning\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "03aa3a49ad3592a9e4e1ab19c6da3e852288caf1", "tree": "eaaa507b6debb862f551d9d3c750b4889e3131ed", "parents": [ "ce6e7fcd43aab1f77e56aa36936dd7d2d05a1ffa" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Tue Jul 21 19:42:03 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Jul 22 21:08:00 2009 +0000" }, "message": "cifs: fix sb-\u003es_maxbytes so that it casts properly to a signed value\n\nThis off-by-one bug causes sendfile() to not work properly. When a task\ncalls sendfile() on a file on a CIFS filesystem, the syscall returns -1\nand sets errno to EOVERFLOW.\n\ndo_sendfile uses s_maxbytes to verify the returned offset of the file.\nThe problem there is that this value is cast to a signed value (loff_t).\nWhen this is done on the s_maxbytes value that cifs uses, it becomes\nnegative and the comparisons against it fail.\n\nEven though s_maxbytes is an unsigned value, it seems that it\u0027s not OK\nto set it in such a way that it\u0027ll end up negative when it\u0027s cast to a\nsigned value. These casts happen in other codepaths besides sendfile\ntoo, but the VFS is a little hard to follow in this area and I can\u0027t\nbe sure if there are other bugs that this will fix.\n\nIt\u0027s not clear to me why s_maxbytes isn\u0027t just declared as loff_t in the\nfirst place, but either way we still need to fix these values to make\nsendfile work properly. This is also an opportunity to replace the magic\nbit-shift values here with the standard #defines for this.\n\nThis fixes the reproducer program I have that does a sendfile and\nwill probably also fix the situation where apache is serving from a\nCIFS share.\n\nAcked-by: Johannes Weiner \u003channes@cmpxchg.org\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "ce6e7fcd43aab1f77e56aa36936dd7d2d05a1ffa", "tree": "d76654cd10a30f12117211924fc4abf68b4fcf3d", "parents": [ "aea1f7964ae6cba5eb419a958956deb9016b3341" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed Jul 22 15:08:58 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Jul 22 21:07:51 2009 +0000" }, "message": "cifs: disable serverino if server doesn\u0027t support it\n\nA recent regression when dealing with older servers. This bug was\nintroduced when we made serverino the default...\n\nWhen the server can\u0027t provide inode numbers, disable it for the mount.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "90a98b2f3f3647fb17667768a348b2b219f2a9f7", "tree": "784f9cf1728176d7f604fede2ce845cfd6bcf31c", "parents": [ "287638b2c533165c3c03dfa15196c2ba583cd287" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Mon Jul 20 13:40:52 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Jul 20 18:24:37 2009 +0000" }, "message": "cifs: free nativeFileSystem field before allocating a new one\n\n...otherwise, we\u0027ll leak this memory if we have to reconnect (e.g. after\nnetwork failure).\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nCC: Stable \u003cstable@kernel.org\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "65bc98b0059360e458aebd208587be44641227c1", "tree": "37db50ecbf083d20a1b4444d49610c7eafd8f8f3", "parents": [ "c2cc49a2f8a479dde96a599646d30b6cc9dbed78" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Jul 10 15:27:25 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Jul 10 15:27:25 2009 +0000" }, "message": "[CIFS] Distinguish posix opens and mkdirs from legacy mkdirs in stats\n\nAcked-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "d0c280d26de9422c9c943f8f486b9830cd9bea70", "tree": "1f2a3471b6d22ace11d954ddd480db539ff69939", "parents": [ "aeaaf253c4dee7ff9af2f3f0595f3bb66964e944" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Jul 09 01:46:44 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Jul 09 23:06:04 2009 +0000" }, "message": "cifs: when ATTR_READONLY is set, only clear write bits on non-directories\n\ncifs: when ATTR_READONLY is set, only clear write bits on non-directories\n\nOn windows servers, ATTR_READONLY apparently either has no meaning or\nserves as some sort of queue to certain applications for unrelated\nbehavior. This MS kbase article has details:\n\nhttp://support.microsoft.com/kb/326549/\n\nDon\u0027t clear the write bits directory mode when ATTR_READONLY is set.\n\nReported-by: pouchat@peewiki.net\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "aeaaf253c4dee7ff9af2f3f0595f3bb66964e944", "tree": "fb575868801fbc8d4da02717c1ef851c9be86785", "parents": [ "0b8f18e358384a52c1ed7fa7129c08e7eaf86bb6" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Jul 09 01:46:39 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Jul 09 23:06:00 2009 +0000" }, "message": "cifs: remove cifsInodeInfo-\u003einUse counter\n\ncifs: remove cifsInodeInfo-\u003einUse counter\n\nIt was purported to be a refcounter of some sort, but was never\nused that way. It never served any purpose that wasn\u0027t served equally well\nby the I_NEW flag.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "0b8f18e358384a52c1ed7fa7129c08e7eaf86bb6", "tree": "ba5292a38d8910c6b3a3303a3c7f88491e1708af", "parents": [ "b77863bfa153e886f9f8faf1a791ba57a36efed0" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Jul 09 01:46:37 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Jul 09 23:05:48 2009 +0000" }, "message": "cifs: convert cifs_get_inode_info and non-posix readdir to use cifs_iget\n\ncifs: convert cifs_get_inode_info and non-posix readdir to use cifs_iget\n\nRather than allocating an inode and filling it out, have\ncifs_get_inode_info fill out a cifs_fattr and call cifs_iget. This means\na pretty hefty reorganization of cifs_get_inode_info.\n\nFor the readdir codepath, add a couple of new functions for filling out\ncifs_fattr\u0027s from different FindFile response infolevels.\n\nFinally, remove cifs_new_inode since there are no more callers.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "b77863bfa153e886f9f8faf1a791ba57a36efed0", "tree": "72b97cd304a58b46ba5cf80e4d4549d289d73594", "parents": [ "3bbeeb3c93a961bd01b969dd4395ecac0c09db8d" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Jul 09 22:51:38 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Jul 09 22:51:38 2009 +0000" }, "message": "[CIFS] update cifs version number\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "3bbeeb3c93a961bd01b969dd4395ecac0c09db8d", "tree": "57f003be4cb7a78094a3806df463844371329a85", "parents": [ "654cf14ac0a71c56c1f0032140c3403382ca076b" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Jul 09 20:02:50 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Jul 09 21:15:10 2009 +0000" }, "message": "cifs: add and use CIFSSMBUnixSetFileInfo for setattr calls\n\ncifs: add and use CIFSSMBUnixSetFileInfo for setattr calls\n\nWhen there\u0027s an open filehandle, SET_FILE_INFO is apparently preferred\nover SET_PATH_INFO. Add a new variant that sets a FILE_UNIX_INFO_BASIC\ninfolevel via SET_FILE_INFO and switch cifs_setattr_unix to use the\nnew call when there\u0027s an open filehandle available.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "654cf14ac0a71c56c1f0032140c3403382ca076b", "tree": "6408ddcd7cbb57c05036ae83fc21881464e67b7b", "parents": [ "01ea95e3b6b16573a491ef98ad63f7a1bdcb504f" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Jul 09 20:02:49 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Jul 09 21:15:06 2009 +0000" }, "message": "cifs: make a separate function for filling out FILE_UNIX_BASIC_INFO\n\ncifs: make a separate function for filling out FILE_UNIX_BASIC_INFO\n\nThe SET_FILE_INFO variant will need to do the same thing here. Break\nthis code out into a separate function that both variants can call.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "01ea95e3b6b16573a491ef98ad63f7a1bdcb504f", "tree": "a060a54d4a3fc1b08534569e2afb8d7c8d5fff37", "parents": [ "c4c1bff64dfff4e6dd0936a0340f56b9284512c8" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Jul 09 20:02:49 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Jul 09 21:15:02 2009 +0000" }, "message": "cifs: rename CIFSSMBUnixSetInfo to CIFSSMBUnixSetPathInfo\n\ncifs: rename CIFSSMBUnixSetInfo to CIFSSMBUnixSetPathInfo\n\n...in preparation of adding a SET_FILE_INFO variant.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "c4c1bff64dfff4e6dd0936a0340f56b9284512c8", "tree": "dca98a2682b172e9c74c0929021e142d2fe281ad", "parents": [ "5ddf1e0ff00fd808c048d0b920784828276cc516" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Jul 09 20:02:48 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Jul 09 21:14:58 2009 +0000" }, "message": "cifs: add pid of initiating process to spnego upcall info\n\ncifs: add pid of initiating process to spnego upcall info\n\nThis will allow the upcall to poke in /proc/\u003cpid\u003e/environ and get\nthe value of the $KRB5CCNAME env var for the process.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "5ddf1e0ff00fd808c048d0b920784828276cc516", "tree": "3510f7e7274700478444ad277f7025519417c7ea", "parents": [ "cc0bad7552308e8905d6ea56e6b7811fa67e716d" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Sun Jul 05 11:01:02 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Jul 08 21:55:45 2009 +0000" }, "message": "cifs: fix regression with O_EXCL creates and optimize away lookup\n\ncifs: fix regression with O_EXCL creates and optimize away lookup\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nTested-by: Shirish Pargaonkar \u003cshirishp@gmail.com\u003e\nCC: Stable Kernel \u003cstable@kernel.org\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "cc0bad7552308e8905d6ea56e6b7811fa67e716d", "tree": "c02a91f13a502f444330b91b36ab12f55f352268", "parents": [ "d960eea974f5e500c0dcb95a934239cc1f481cfd" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Jun 25 00:56:52 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Jul 01 21:26:42 2009 +0000" }, "message": "cifs: add new cifs_iget function and convert unix codepath to use it\n\ncifs: add new cifs_iget function and convert unix codepath to use it\n\nIn order to unify some codepaths, introduce a common cifs_fattr struct\nfor storing inode attributes. The different codepaths (unix, legacy,\nnormal, etc...) can fill out this struct with inode info. It can then be\npassed as an arg to a common set of routines to get and update inodes.\n\nAdd a new cifs_iget function that uses iget5_locked to identify inodes.\nThis will compare inodes based on the uniqueid value in a cifs_fattr\nstruct.\n\nRather than filling out an already-created inode, have\ncifs_get_inode_info_unix instead fill out cifs_fattr and hand that off\nto cifs_iget. cifs_iget can then properly look for hardlinked inodes.\n\nOn the readdir side, add a new cifs_readdir_lookup function that spawns\npopulated dentries. Redefine FILE_UNIX_INFO so that it\u0027s basically a\nFILE_UNIX_BASIC_INFO that has a few fields wrapped around it. This\nallows us to more easily use the same function for filling out the fattr\nas the non-readdir codepath.\n\nWith this, we should then have proper hardlink detection and can\neventually get rid of some nasty CIFS-specific hacks for handing them.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "f0a71eb820596bd8f6abf64beb4cb181edaa2341", "tree": "f46ee612397a696ff1f601f9c5960a0eb5ed0c26", "parents": [ "4075ea8c54a7506844a69f674990241e7766357b" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Sat Jun 27 07:04:55 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat Jun 27 23:46:43 2009 +0000" }, "message": "cifs: fix fh_mutex locking in cifs_reopen_file\n\nFixes a regression caused by commit a6ce4932fbdbcd8f8e8c6df76812014351c32892\n\nWhen this lock was converted to a mutex, the locks were turned into\nunlocks and vice-versa.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nAcked-by: Shirish Pargaonkar \u003cshirishp@us.ibm.com\u003e\nCc: Stable Tree \u003cstable@kernel.org\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "71a394faaad07090af5de5c075ec2f5bca0fbb35", "tree": "9e4f67deea3ecfc6bb571a930c16c6d6e5b10f3f", "parents": [ "ad8034f19792736db5c259103c2eaaf72887bbb4" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Jun 26 04:07:18 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Jun 26 04:07:18 2009 +0000" }, "message": "[CIFS] remove unknown mount option warning message\n\nJeff\u0027s previous patch which removed the unneeded rw/ro\nparsing can cause a minor warning in dmesg (about the\nunknown rw or ro mount option) at mount time. This\npatch makes cifs ignore them in kernel to remove the warning\n(they are already handled in the mount helper and VFS).\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "ad8034f19792736db5c259103c2eaaf72887bbb4", "tree": "ad631c38d2d66c6d3ca57cdae471560461cec119", "parents": [ "0f3bc09ee1b7fcadd5bfdc5ed2e1643f658fe23d" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Jun 26 03:25:49 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri Jun 26 03:25:49 2009 +0000" }, "message": "[CIFS] remove bkl usage from umount begin\n\nThe lock_kernel call moved into the fs for umount_begin\nis not needed. This adds a check to make sure we don\u0027t\ncall umount_begin twice on the same fs.\n\numount_begin for cifs is probably not needed and\nmay eventually be able to be removed, but in\nthe meantime this smaller patch is safe and\ngets rid of the bkl from this path which provides\nsome benefit.\n\nAcked-by: Jeff Layton \u003credhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "0f3bc09ee1b7fcadd5bfdc5ed2e1643f658fe23d", "tree": "fdbbdeeefcc1b480990cb6a687387c1e232a39e0", "parents": [ "f46c7234e472ceee39afea4fb5a4365843e1850a" ], "author": { "name": "Suresh Jayaraman", "email": "sjayaraman@suse.de", "time": "Thu Jun 25 18:12:34 2009 +0530" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Jun 25 19:12:57 2009 +0000" }, "message": "cifs: Fix incorrect return code being printed in cFYI messages\n\nFreeXid() along with freeing Xid does add a cifsFYI debug message that\nprints rc (return code) as well. In some code paths where we set/return\nerror code after calling FreeXid(), incorrect error code is being\nprinted when cifsFYI is enabled.\n\nThis could be misleading in few cases. For eg.\nIn cifs_open() if cifs_fill_filedata() returns a valid pointer to\ncifsFileInfo, FreeXid() prints rc\u003d-13 whereas 0 is actually being\nreturned. Fix this by setting rc before calling FreeXid().\n\nBasically convert\n\nFreeXid(xid);\t\t\trc \u003d -ERR;\nreturn -ERR;\t\t\u003d\u003e\tFreeXid(xid);\n\t\t\t\treturn rc;\n\n[Note that Christoph would like to replace the GetXid/FreeXid\ncalls, which are primarily used for debugging. This seems\nlike a good longer term goal, but although there is an\nalternative tracing facility, there are no examples yet\navailable that I know of that we can use (yet) to\nconvert this cifs function entry/exit logging, and for\ncreating an identifier that we can use to correlate\nall dmesg log entries for a particular vfs operation\n(ie identify all log entries for a particular vfs\nrequest to cifs: e.g. a particular close or read or write\nor byte range lock call ... and just using the thread id\nis harder). Eventually when a replacement\nfor this is available (e.g. when NFS switches over and various\nsamples to look at in other file systems) we can remove the\nGetXid/FreeXid macro but in the meantime multiple people\nuse this run time configurable logging all the time\nfor debugging, and Suresh\u0027s patch fixes a problem\nwhich made it harder to notice some low\nmemory problems in the log so it is worthwhile\nto fix this problem until a better logging\napproach is able to be used]\n\nAcked-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Suresh Jayaraman \u003csjayaraman@suse.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "f46c7234e472ceee39afea4fb5a4365843e1850a", "tree": "9f72939ab67744212b15a2a114645e63da753aa4", "parents": [ "6debdbc0ba6253ac519cd5a3d22e30f1f9f1dd12" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Jun 25 03:04:20 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Jun 25 03:07:48 2009 +0000" }, "message": "[CIFS] cleanup asn handling for ntlmssp\n\nAlso removes obsolete distinction between rawntlmssp and ntlmssp (in asn/SPNEGO)\nsince as jra noted we can always send raw ntlmssp in session setup now.\n\nremove check for experimental runtime flag (/proc/fs/cifs/Experimental) in\nntlmssp path.\n\nReviewed-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "6debdbc0ba6253ac519cd5a3d22e30f1f9f1dd12", "tree": "523ef732a835d25a30d997bbff6fcf8321360857", "parents": [ "6459340cfcc6f6d165b27c3dd955aeb55a1b73d3" ], "author": { "name": "Simo Leone", "email": "simo@archlinux.org", "time": "Thu Jun 25 02:44:43 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Jun 25 02:44:43 2009 +0000" }, "message": "[CIFS] Copy struct *after* setting the port, instead of before.\n\nAcked-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Simo Leone \u003csimo@archlinux.org\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "6459340cfcc6f6d165b27c3dd955aeb55a1b73d3", "tree": "1dd835d4931f2f02c62b7ac6f4520858ef0f5580", "parents": [ "b48a485884b5afb3e33b1871bcbd246b67491923" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Jun 25 00:56:55 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Jun 25 02:33:01 2009 +0000" }, "message": "cifs: remove rw/ro options\n\ncifs: remove rw/ro options\n\nThese options are handled at the VFS layer. They only ever set the\noption in the smb_vol struct. Nothing was ever done with them afterward\nanyway.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "b48a485884b5afb3e33b1871bcbd246b67491923", "tree": "d04de55957183d858defc8f6ab1d1a79af24a5a5", "parents": [ "681bf72e4893a187cf6b6b62c08fc193f81c8c2f" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Jun 25 00:56:54 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Jun 25 02:32:57 2009 +0000" }, "message": "cifs: fix problems with earlier patches\n\ncifs: fix problems with earlier patches\n\ncifs_show_address hasn\u0027t been introduced yet, and fix a typo that was\nsilently fixed by a later patch in the series.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "681bf72e4893a187cf6b6b62c08fc193f81c8c2f", "tree": "5c580474a21edf38140407bd42f28853e9354e25", "parents": [ "268875b9d1dd1bf0b523c59e736da9bc20c8ce1f" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Jun 11 10:27:31 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Jun 25 01:14:36 2009 +0000" }, "message": "cifs: have cifs parse scope_id out of IPv6 addresses and use it\n\nThis patch has CIFS look for a \u0027%\u0027 in an IPv6 address. If one is\npresent then it will try to treat that value as a numeric interface\nindex suitable for stuffing into the sin6_scope_id field.\n\nThis should allow people to mount servers on IPv6 link-local addresses.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nAcked-by: David Holder \u003cdavid@erion.co.uk\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "268875b9d1dd1bf0b523c59e736da9bc20c8ce1f", "tree": "9d2a55c5847aadfbfc1c50617dc469dd40716900", "parents": [ "361ea1ae5451040cd254eee0b6df64581080b2cc" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Jun 25 00:29:21 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu Jun 25 00:29:21 2009 +0000" }, "message": "[CIFS] Do not send tree disconnect if session is already disconnected\n\nNoticed this when tree connect timed out (due to Samba server crash) -\nwe try to send a tree disconnect for a tid that does not exist\nsince we don\u0027t have a valid tree id yet. This checks that the\nsession is valid before sending the tree disconnect to handle\nthis case.\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "361ea1ae5451040cd254eee0b6df64581080b2cc", "tree": "7ee6e493c34cf9f7d1509571af925ef56a46b90a", "parents": [ "61f98ffd74254a95871168bd5a6646b4f3002e31" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Jun 15 13:46:12 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Jun 15 13:46:12 2009 +0000" }, "message": "[CIFS] Fix build break\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "61f98ffd74254a95871168bd5a6646b4f3002e31", "tree": "c66a4df0ccc4bedc58c7b45125a99c0ac61bc4da", "parents": [ "b70b92e41d95fd906f05f6e98f61209201495fa7" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Jun 11 10:27:32 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon Jun 15 13:42:30 2009 +0000" }, "message": "cifs: display scopeid in /proc/mounts\n\nMove address display into a new function and display the scopeid as part\nof the address in /proc/mounts.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "1e68b2b2756fc3488ecbade5ad5f13302b3aaafc", "tree": "3ee4b36a1a00d449126050ce3bb1758ba88d7c2f", "parents": [ "340481a36498bf3fe404bcecb2e2d6188e950bff" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Jun 11 10:27:30 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat Jun 13 08:17:30 2009 +0000" }, "message": "cifs: add new routine for converting AF_INET and AF_INET6 addrs\n\n...to consolidate some logic used in more than one place.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "340481a36498bf3fe404bcecb2e2d6188e950bff", "tree": "d5e8cc4dd7bb7d111f29170642170ea921017b00", "parents": [ "8616e0fc1e27295316f9821a883f0e9fa6f8200f" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Jun 11 10:27:29 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat Jun 13 08:17:30 2009 +0000" }, "message": "cifs: have cifs_show_options show forceuid/forcegid options\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "8616e0fc1e27295316f9821a883f0e9fa6f8200f", "tree": "99b3b5c887cb8d4ba244c51fdc9d5ba95102f6b3", "parents": [ "f3ad116588151b3371ae4e092290e4f48e62b8bb" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu Jun 11 10:27:28 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat Jun 13 08:17:30 2009 +0000" }, "message": "cifs: remove unneeded NULL checks from cifs_show_options\n\nshow_options is always called with the namespace_sem held. Therefore we\ndon\u0027t need to worry about the vfsmount being NULL, or it vanishing while\nthe function is running. By the same token, there\u0027s no need to worry\nabout the superblock, tcon, smb or tcp sessions being NULL on entry.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "6cfd0148425e528b859b26e436b01f23f6926224", "tree": "60e3257053554ff198fe5825e6f12a00c3b4422a", "parents": [ "a9e220f8322e2b0e0b8903fe00265461cffad3f0" ], "author": { "name": "Christoph Hellwig", "email": "hch@lst.de", "time": "Tue May 05 15:40:36 2009 +0200" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Thu Jun 11 21:36:07 2009 -0400" }, "message": "push BKL down into -\u003eput_super\n\nMove BKL into -\u003eput_super from the only caller. A couple of\nfilesystems had trivial enough -\u003eput_super (only kfree and NULLing of\ns_fs_info + stuff in there) to not get any locking: coda, cramfs, efs,\nhugetlbfs, omfs, qnx4, shmem, all others got the full treatment. Most\nof them probably don\u0027t need it, but I\u0027d rather sort that out individually.\nPreferably after all the other BKL pushdowns in that area.\n\n[AV: original used to move lock_super() down as well; these changes are\nremoved since we don\u0027t do lock_super() at all in generic_shutdown_super()\nnow]\n[AV: fuse, btrfs and xfs are known to need no damn BKL, exempt]\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "9393bd07cf218ca51d0e627653f906a9d76a9131", "tree": "402205fe6336028df48cd65b31da0482f0b0bb41", "parents": [ "589ff870ed60a9ebdd5ec99ec3f5afe1282fe151" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat Apr 18 13:58:15 2009 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Thu Jun 11 21:36:01 2009 -0400" }, "message": "switch follow_down()\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "61b6bc525a34931bb73e4c95bfe009cd9572a288", "tree": "4c60f5233b72c7ab1a4a4e6e123d49ba2de3c13c", "parents": [ "58f7f68f228c3aba2ba4468d92e2cec35724ba2e" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed Jun 10 10:04:58 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Jun 10 18:34:35 2009 +0000" }, "message": "cifs: remove never-used in6_addr option\n\nThis option was never used to my knowledge. Remove it before someone\ndoes...\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "58f7f68f228c3aba2ba4468d92e2cec35724ba2e", "tree": "3afd0bfcfa97ebd76154f10cfc6dae4d839589d4", "parents": [ "f0472d0ec89bef2ea4432828c3daa1b26ef569aa" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed Jun 10 09:57:55 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed Jun 10 15:39:14 2009 +0000" }, "message": "cifs: add addr\u003d mount option alias for ip\u003d\n\nWhen you look in /proc/mounts, the address of the server gets displayed\nas \"addr\u003d\". That\u0027s really a better option to use anyway since it\u0027s more\ngeneric. What if we eventually want to support non-IP transports? It\nalso makes CIFS option consistent with the NFS option of the same name.\n\nBegin the migration to that option name by adding an alias for ip\u003d\ncalled addr\u003d.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "f0472d0ec89bef2ea4432828c3daa1b26ef569aa", "tree": "3241e426eee1e7a3233f9649e779984aa18f9b98", "parents": [ "4ae1507f6d266d0cc3dd36e474d83aad70fec9e4" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat Jun 06 21:09:39 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat Jun 06 21:09:39 2009 +0000" }, "message": "[CIFS] Add mention of new mount parm (forceuid) to cifs readme\n\nAlso update fs/cifs/CHANGES\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "4ae1507f6d266d0cc3dd36e474d83aad70fec9e4", "tree": "ae41fd204f772ecc8f9adc2e727a9eb01999784e", "parents": [ "50b64e3b77d569c217a48e078cd565dbd6462ad0" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Sun May 24 18:45:15 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat Jun 06 21:03:27 2009 +0000" }, "message": "cifs: make overriding of ownership conditional on new mount options\n\nWe have a bit of a problem with the uid\u003d option. The basic issue is that\nit means too many things and has too many side-effects.\n\nIt\u0027s possible to allow an unprivileged user to mount a filesystem if the\nuser owns the mountpoint, /bin/mount is setuid root, and the mount is\nset up in /etc/fstab with the \"user\" option.\n\nWhen doing this though, /bin/mount automatically adds the \"uid\u003d\" and\n\"gid\u003d\" options to the share. This is fortunate since the correct uid\u003d\noption is needed in order to tell the upcall what user\u0027s credcache to\nuse when generating the SPNEGO blob.\n\nOn a mount without unix extensions this is fine -- you generally will\nwant the files to be owned by the \"owner\" of the mount. The problem\ncomes in on a mount with unix extensions. With those enabled, the\nuid/gid options cause the ownership of files to be overriden even though\nthe server is sending along the ownership info.\n\nThis means that it\u0027s not possible to have a mount by an unprivileged\nuser that shows the server\u0027s file ownership info. The result is also\ninode permissions that have no reflection at all on the server. You\nsimply cannot separate ownership from the mode in this fashion.\n\nThis behavior also makes MultiuserMount option less usable. Once you\npass in the uid\u003d option for a mount, then you can\u0027t use unix ownership\ninfo and allow someone to share the mount.\n\nWhile I\u0027m not thrilled with it, the only solution I can see is to stop\nmaking uid\u003d/gid\u003d force the overriding of ownership on mounts, and to add\nnew mount options that turn this behavior on.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "50b64e3b77d569c217a48e078cd565dbd6462ad0", "tree": "2001150788641aad1f4824046d336f0b5d4dd32c", "parents": [ "b96d31a62f714566fa6420851b3bb3615c796322" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Tue Jun 02 06:55:20 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue Jun 02 15:45:40 2009 +0000" }, "message": "cifs: fix IPv6 address length check\n\nFor IPv6 the userspace mount helper sends an address in the \"ip\u003d\"\noption. This check fails if the length is \u003e 35 characters. I have no\nidea where the magic 35 character limit came from, but it\u0027s clearly not\nenough for IPv6. Fix it by making it use the INET6_ADDRSTRLEN #define.\n\nWhile we\u0027re at it, use the same #define for the address length in SPNEGO\nupcalls.\n\nReported-by: Charles R. Anderson \u003ccra@wpi.edu\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "b96d31a62f714566fa6420851b3bb3615c796322", "tree": "5d73eab49e4e745a837e0e4192ac022a570482aa", "parents": [ "1bf4072da67c14d6b02cfeef02212aa5a6211df2" ], "author": { "name": "Christoph Hellwig", "email": "hch@infradead.org", "time": "Wed May 27 09:37:33 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 28 18:41:56 2009 +0000" }, "message": "cifs: clean up set_cifs_acl interfaces\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nReviewed-by: Shirish Pargaonkar \u003cshirishp@us.ibm.com\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "1bf4072da67c14d6b02cfeef02212aa5a6211df2", "tree": "610f12379929abda22be85fda824efc55626f60b", "parents": [ "c5077ec42303e07c2c685b0f6cb8eee0f2c7751c" ], "author": { "name": "Christoph Hellwig", "email": "hch@infradead.org", "time": "Wed May 27 09:37:33 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 28 17:08:02 2009 +0000" }, "message": "cifs: reorganize get_cifs_acl\n\nThus spake Christoph:\n\n\"But this whole set_cifs_acl function is a real mess anyway and needs\nsome splitting up.\"\n\nWith this change too, it\u0027s possible to call acl_to_uid_mode() with a\nNULL inode pointer. That (or something close to it) will eventually be\nnecessary when cifs_get_inode_info is reorganized.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nReviewed-by: Shirish Pargaonkar \u003cshirishp@us.ibm.com\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "c5077ec42303e07c2c685b0f6cb8eee0f2c7751c", "tree": "013c3f682f2c870bb8a7818142bf725c2e9407fa", "parents": [ "a0c9217f64ee3cd1e534966da8c5f05768e1ab09" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 28 15:09:04 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 28 15:09:04 2009 +0000" }, "message": "[CIFS] Update readme to indicate change to default mount (serverino)\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "a0c9217f64ee3cd1e534966da8c5f05768e1ab09", "tree": "c38584b9ca0376028b3273ce9515e84a3ddb8aec", "parents": [ "bd433d4cf4d8593a5f1764776b91f1794fce5a77" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed May 27 15:40:47 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 28 15:04:17 2009 +0000" }, "message": "cifs: make serverino the default when mounting\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "bd433d4cf4d8593a5f1764776b91f1794fce5a77", "tree": "49656dd0ef08792dfd86db25216a4d04211b923f", "parents": [ "c4a2c08db7d976c2e23a97da5d69ec7c9701034d" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed May 27 09:37:34 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 28 14:57:29 2009 +0000" }, "message": "cifs: rename cifs_iget to cifs_root_iget\n\nThe current cifs_iget isn\u0027t suitable for anything but the root inode.\nRename it with a more appropriate name.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "c4a2c08db7d976c2e23a97da5d69ec7c9701034d", "tree": "803c588c4f60a50bbc3cf6465cb380c8ee95e3cd", "parents": [ "07119a4df8c8c77d888f2f46964ea9512ea84ff8" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed May 27 09:37:33 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 28 14:57:20 2009 +0000" }, "message": "cifs: make cnvrtDosUnixTm take a little-endian args and an offset\n\nThe callers primarily end up converting the args from le anyway. Also,\nmost of the callers end up needing to add an offset to the result. The\nexception to these rules is cnvrtDosCifsTm, but there are no callers of\nthat function, so we might as well remove it.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "07119a4df8c8c77d888f2f46964ea9512ea84ff8", "tree": "546a3c2e66b60db890421c83106d5f6329e9ffce", "parents": [ "f55ed1a83d099f275c9560ad7d4c4700d1e54bdd" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Wed May 27 09:37:33 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 28 14:32:31 2009 +0000" }, "message": "cifs: have cifs_NTtimeToUnix take a little-endian arg\n\n...and just have the function call le64_to_cpu.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "f55ed1a83d099f275c9560ad7d4c4700d1e54bdd", "tree": "863038ab400c725cd7be24ff8782a313ed6a20a0", "parents": [ "46a7574caf5bc533c24b315800ed323c187614f5" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Tue May 26 16:28:11 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue May 26 21:10:55 2009 +0000" }, "message": "cifs: tighten up default file_mode/dir_mode\n\nThe current default file mode is 02767 and dir mode is 0777. This is\nextremely \"loose\". Given that CIFS is a single-user protocol, these\npermissions allow anyone to use the mount -- in effect, giving anyone on\nthe machine access to the credentials used to mount the share.\n\nChange this by making the default permissions restrict write access to\nthe default owner of the mount. Give read and execute permissions to\neveryone else. These are the same permissions that VFAT mounts get by\ndefault so there is some precedent here.\n\nNote that this patch also removes the mandatory locking flags from the\ndefault file_mode. After having looked at how these flags are used by\nthe kernel, I don\u0027t think that keeping them as the default offers any\nreal benefit. That flag combination makes it so that the kernel enforces\nmandatory locking.\n\nSince the server is going to do that for us anyway, I don\u0027t think we\nwant the client to enforce this by default on applications that just\nwant advisory locks. Anyone that does want this behavior can always\nenable it by setting the file_mode appropriately.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "46a7574caf5bc533c24b315800ed323c187614f5", "tree": "2afb4ee1af82a06fa16a9ae4b14dedeec7addbce", "parents": [ "e2a1b9ee2335c35e0e34c88a024481b194b3c9cc" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Sun May 24 18:45:17 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue May 26 21:09:14 2009 +0000" }, "message": "cifs: fix artificial limit on reading symlinks\n\nThere\u0027s no reason to limit the size of a symlink that we can read to\n4000 bytes. That may be nowhere near PATH_MAX if the server is sending\nUCS2 strings. CIFS should be able to read in a symlink up to the size of\nthe buffer. The size of the header has already been accounted for when\ncreating the slabcache, so CIFSMaxBufSize should be the correct size to\npass in.\n\nFixes samba bug #6384.\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "8db14ca12569fe885694bd3d5ff84c2d973d3cb0", "tree": "c1648afbd9c6e93b6b48f230c5c7dbb3d0184a2b", "parents": [ "a15ae93ff7c710191362978453f306943808298d" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat May 23 18:57:25 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat May 23 18:57:25 2009 +0000" }, "message": "[CIFS] Avoid open on possible directories since Samba now rejects them\n\nSmall change (mostly formatting) to limit lookup based open calls to\nfile create only.\n\nAfter discussion yesteday on samba-technical about the posix lookup\nregression, and looking at a problem with cifs posix open to one\nparticular Samba version, Jeff and JRA realized that Samba server\u0027s\nbehavior changed in this area (posix open behavior on files vs.\ndirectories). To make this behavior consistent, JRA just made a\nfix to Samba server to alter how it handles open of directories (now\nreturning the equivalent of EISDIR instead of success). Since we don\u0027t\nknow at lookup time whether the inode is a directory or file (and\nthus whether posix open will succeed with most current Samba server),\nthis change avoids the posix open code on lookup open (just issues\nposix open on creates). This gets the semantic benefits we want\n(atomicity, posix byte range locks, improved write semantics on newly\ncreated files) and file create still is fast, and we avoid the problem\nthat Jeff noticed yesterday with \"openat\" (and some open directory\ncalls) of non-cached directories to one version of Samba server, and\nwill work with future Samba versions (which include the fix jra just\npushed into Samba server). I confirmed this approach with jra\nyesterday and with Shirish today.\n\nPosix open is only called (at lookup time) for file create now.\nFor opens (rather than creates), because we do not know if it\nis a file or directory yet, and current Samba no longer allows\nus to do posix open on dirs, we could end up wasting an open call\non what turns out to be a dir. For file opens, we wait to call posix\nopen till cifs_open. It could be added here (lookup) in the future\nbut the performance tradeoff of the extra network request when EISDIR\nor EACCES is returned would have to be weighed against the 50%\nreduction in network traffic in the other paths.\n\nReviewed-by: Shirish Pargaonkar \u003cshirishp@us.ibm.com\u003e\nTested-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nCC: Jeremy Allison \u003cjra@samba.org\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "703a3b8e5c01cf6fb33c6d8dc99905f889a4e992", "tree": "56b6f7bd0379efc28fd488b15fe121a03d092f3d", "parents": [ "5805977e63a36ad56594a623f3bd2bebcb7db233" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 21 22:21:53 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 21 22:38:08 2009 +0000" }, "message": "[CIFS] fix posix open regression\n\nPosix open code was not properly adding the file to the\nlist of open files. Fix allocating cifsFileInfo\nmore than once, and adding twice to flist and tlist.\nAlso fix mode setting to be done in one place in these\npaths.\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\nReviewed-by: Shirish Pargaonkar \u003cshirishp@us.ibm.com\u003e\nTested-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nTested-by: Luca Tettamanti \u003ckronos.it@gmail.com\u003e\n" }, { "commit": "8b6427a2a8f7dd43e9208fb33a3b116d66db4979", "tree": "51b3bcb46a36f3bbc9bfb5ea61cdf600b2233897", "parents": [ "b41a080fa9f157d223c782ec3571cf46e34e91d6" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Tue May 19 09:57:03 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Tue May 19 15:31:20 2009 +0000" }, "message": "cifs: fix pointer initialization and checks in cifs_follow_symlink (try #4)\n\nThis is the third respin of the patch posted yesterday to fix the error\nhandling in cifs_follow_symlink. It also includes a fix for a bogus NULL\npointer check in CIFSSMBQueryUnixSymLink that Jeff Moyer spotted.\n\nIt\u0027s possible for CIFSSMBQueryUnixSymLink to return without setting\ntarget_path to a valid pointer. If that happens then the current value\nto which we\u0027re initializing this pointer could cause an oops when it\u0027s\nkfree\u0027d.\n\nThis patch is a little more comprehensive than the last patches. It\nreorganizes cifs_follow_link a bit for (hopefully) better readability.\nIt should also eliminate the uneeded allocation of full_path on servers\nwithout unix extensions (assuming they can get to this point anyway, of\nwhich I\u0027m not convinced).\n\nOn a side note, I\u0027m not sure I agree with the logic of enabling this\nquery even when unix extensions are disabled on the client. It seems\nlike that should disable this as well. But, changing that is outside the\nscope of this fix, so I\u0027ve left it alone for now.\n\nReported-by: Jeff Moyer \u003cjmoyer@redhat.com\u003e\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nReviewed-by: Jeff Moyer \u003cjmoyer@redhat.com\u003e\nReviewed-by: Christoph Hellwig \u003chch@inraded.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "d8e2f53ac99f4ce7d63807a84f98d1b80df598cf", "tree": "2b956d4bf048e9b72b99a84f589baedbdf6f757c", "parents": [ "45d447406a19cbfd42720f066f156f4eb9d68801" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Thu May 14 07:46:59 2009 -0400" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Thu May 14 13:55:32 2009 +0000" }, "message": "cifs: fix error handling in parse_DFS_referrals\n\ncifs_strndup_from_ucs returns NULL on error, not an ERR_PTR\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "6f5bbff9a1b7d6864a495763448a363bbfa96324", "tree": "0067dca46f40def1c55541c34c262e06aeb8c4c8", "parents": [ "74dbbdd7fdc11763f4698d2f3e684cf4446951e6" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed May 06 01:34:22 2009 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat May 09 10:49:40 2009 -0400" }, "message": "Convert obvious places to deactivate_locked_super()\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "67e55205ec55cc7899f1d783f217961596798419", "tree": "45dd7351f9721a46ecb448f0571fb10db3259147", "parents": [ "091bf7624d1c90cec9e578a18529f615213ff847" ], "author": { "name": "Alessio Igor Bogani", "email": "abogani@texware.it", "time": "Fri Apr 24 09:06:53 2009 +0200" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sat May 09 10:49:38 2009 -0400" }, "message": "vfs: umount_begin BKL pushdown\n\nPush BKL down into -\u003eumount_begin()\n\nSigned-off-by: Alessio Igor Bogani \u003cabogani@texware.it\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "90e4ee5d311d4e0729daa676b1d7f754265b5874", "tree": "d1731b21e054dab9edde4f6132e8999d9f269063", "parents": [ "ac68392460ffefed13020967bae04edc4d3add06" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 08 03:04:30 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 08 03:04:30 2009 +0000" }, "message": "[CIFS] Fix double list addition in cifs posix open code\n\nRemove adding open file entry twice to lists in the file\nDo not fill file info twice in case of posix opens and creates\n\nSigned-off-by: Shirish Pargaonkar \u003cshirishp@us.ibm.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "ac68392460ffefed13020967bae04edc4d3add06", "tree": "9b5e43db83752e8927ee5e449dfafefe12ae1f7d", "parents": [ "844823cb822932d2c599abf38692e3d6a5b5a320" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed May 06 04:16:04 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed May 06 04:16:04 2009 +0000" }, "message": "[CIFS] Allow raw ntlmssp code to be enabled with sec\u003dntlmssp\n\nOn mount, \"sec\u003dntlmssp\" can now be specified to allow\n\"rawntlmssp\" security to be enabled during\nCIFS session establishment/authentication (ntlmssp used to\nrequire specifying krb5 which was counterintuitive).\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "844823cb822932d2c599abf38692e3d6a5b5a320", "tree": "cde5759733879247cdc88cd21d0ca404f8b77033", "parents": [ "0b3cc858003b79b6c66ad79415ead907cbe4074e" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed May 06 00:48:30 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Wed May 06 00:48:30 2009 +0000" }, "message": "[CIFS] Fix SMB uid in NTLMSSP authenticate request\n\nWe were not setting the SMB uid in NTLMSSP authenticate\nrequest which could lead to INVALID_PARAMETER error\non 2nd session setup.\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "0b3cc858003b79b6c66ad79415ead907cbe4074e", "tree": "88449edd4cdb48502ddd98c2c71388f99ed1b59b", "parents": [ "24d35add2bd09a427cacb7a39e14f3e47ed4d766" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon May 04 08:37:12 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Mon May 04 08:37:12 2009 +0000" }, "message": "[CIFS] NTLMSSP reenabled after move from connect.c to sess.c\n\nThe NTLMSSP code was removed from fs/cifs/connect.c and merged\n(75% smaller, cleaner) into fs/cifs/sess.c\n\nAs with the old code it requires that cifs be built with\nCONFIG_CIFS_EXPERIMENTAL, the /proc/fs/cifs/Experimental flag\nmust be set to 2, and mount must turn on extended security\n(e.g. with sec\u003dkrb5).\n\nAlthough NTLMSSP encapsulated in SPNEGO is not enabled yet,\n\"raw\" ntlmssp is common and useful in some cases since it\noffers more complete security negotiation, and is the\ndefault way of negotiating security for many Windows systems.\nSPNEGO encapsulated NTLMSSP will be able to reuse the same\ncode.\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "24d35add2bd09a427cacb7a39e14f3e47ed4d766", "tree": "197105d49e9808b01387d0de45a63285d9c67f61", "parents": [ "989c7e512f4dc976b10803ab0c449bdaaf3eaabd" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat May 02 05:40:39 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat May 02 05:40:39 2009 +0000" }, "message": "[CIFS] Remove sparse warning\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "989c7e512f4dc976b10803ab0c449bdaaf3eaabd", "tree": "05cdba5c81df7ba19d21d492e506962711262aa1", "parents": [ "afe48c31ea5c74eaac58621ce1c85ae8187c4383" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat May 02 05:32:20 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat May 02 05:32:20 2009 +0000" }, "message": "[CIFS] remove checkpatch warning\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "afe48c31ea5c74eaac58621ce1c85ae8187c4383", "tree": "6a6e58bbc1bc6725ec28b8c87b6d9113f375651e", "parents": [ "341060273232a2df0d1a7fa53abc661fcf22747c" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat May 02 05:25:46 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat May 02 05:25:46 2009 +0000" }, "message": "[CIFS] Fix final user of old string conversion code\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "341060273232a2df0d1a7fa53abc661fcf22747c", "tree": "60af81806205be40f3f60773b98bebc33ce97153", "parents": [ "2edd6c5b0517b9131ede9e74cb121898ccd73042" ], "author": { "name": "Jeff Layton", "email": "jlayton@redhat.com", "time": "Sat May 02 04:59:34 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat May 02 04:59:34 2009 +0000" }, "message": "[CIFS] remove cifs_strfromUCS_le\n\nSigned-off-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "2edd6c5b0517b9131ede9e74cb121898ccd73042", "tree": "0c811f33cd5dfd20d48a0585fb744e9f937823f2", "parents": [ "051a2a0d3242b448281376bb63cfa9385e0b6c68" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat May 02 04:55:39 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Sat May 02 04:55:39 2009 +0000" }, "message": "[CIFS] NTLMSSP support moving into new file, old dead code removed\n\nRemove dead NTLMSSP support from connect.c prior to addition of\nthe new code to replace it.\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "051a2a0d3242b448281376bb63cfa9385e0b6c68", "tree": "f0fd4e09526dfb72a4d065fa5144f5dc0f5e99b1", "parents": [ "e836f015b5c07da2f95a441274ef0a788ce17f80" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 01 16:21:04 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 01 16:25:15 2009 +0000" }, "message": "[CIFS] Fix endian conversion of vcnum field\n\nWhen multiply mounting from the same client to the same server, with\ndifferent userids, we create a vcnum which should be unique if\npossible (this is not the same as the smb uid, which is the handle\nto the security context). We were not endian converting additional\n(beyond the first which is zero) vcnum properly.\n\nCC: Stable \u003cstable@kernel.org\u003e\nAcked-by: Shirish Pargaonkar \u003cshirishp@us.ibm.com\u003e\nAcked-by: Jeff Layton \u003cjlayton@redhat.com\u003e\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "e836f015b5c07da2f95a441274ef0a788ce17f80", "tree": "6a8503c183b6b2896a2a63d9c2c6de165c10189d", "parents": [ "0e0d2cf32743c660aab20e40aeb2155c06a256db" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 01 16:20:35 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 01 16:20:35 2009 +0000" }, "message": "[CIFS] Remove trailing whitespace\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" }, { "commit": "0e0d2cf32743c660aab20e40aeb2155c06a256db", "tree": "94dfb57ad091f283172628a3bba754e1562ded7d", "parents": [ "e14b2fe1e64d3e4bd2f328ff9d1969f318f55954" ], "author": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 01 05:27:32 2009 +0000" }, "committer": { "name": "Steve French", "email": "sfrench@us.ibm.com", "time": "Fri May 01 05:27:32 2009 +0000" }, "message": "[CIFS] Remove sparse endian warnings\n\nRemoves two sparse CHECK_ENDIAN warnings from Jeffs earlier patch,\nand removes the dead readlink code (after noting where in\nfindfirst we will need to add something like that in the future\nto handle the newly discovered unexpected error on FindFirst of NTFS symlinks.\n\nSigned-off-by: Steve French \u003csfrench@us.ibm.com\u003e\n" } ], "next": "e14b2fe1e64d3e4bd2f328ff9d1969f318f55954" }