bpo-32277: Fix exception raised from chmod(..., follow_symlinks=False) (#4797)

233ef249 · Anthony Sottile · Serhiy Storchaka · 02a0a192 · 233ef249 · 233ef249
Commit 233ef249 authored Dec 14, 2017 by Anthony Sottile Committed by Serhiy Storchaka Dec 14, 2017
Showing with 4 additions and 0 deletions

Misc/NEWS.d/next/Library/2017-12-11-09-53-14.bpo-32277.jkKiVC.rst ...S.d/next/Library/2017-12-11-09-53-14.bpo-32277.jkKiVC.rst +3 -0

Modules/posixmodule.c Modules/posixmodule.c +1 -0

No files found.
--- a/Misc/NEWS.d/next/Library/2017-12-11-09-53-14.bpo-32277.jkKiVC.rst
+++ b/Misc/NEWS.d/next/Library/2017-12-11-09-53-14.bpo-32277.jkKiVC.rst
+Raise ``NotImplementedError`` instead of ``SystemError`` on platforms where
+``chmod(..., follow_symlinks=False)`` is not supported.  Patch by Anthony
+Sottile.
--- a/Modules/posixmodule.c
+++ b/Modules/posixmodule.c
@@ -2827,6 +2827,7 @@ os_chmod_impl(PyObject *module, path_t *path, int mode, int dir_fd,
                                                   dir_fd, follow_symlinks);
            else
                follow_symlinks_specified("chmod", follow_symlinks);
+            return NULL;
        }
        else
 #endif