diff --git a/magit-svn.el b/magit-svn.el
index 1ad0db45..e38f7667 100644
--- a/magit-svn.el
+++ b/magit-svn.el
@@ -159,8 +159,8 @@ If USE-CACHE is non nil, use the cached information."
 (define-key magit-svn-map (kbd "r") 'magit-svn-rebase)
 (define-key magit-svn-map (kbd "c") 'magit-svn-dcommit)
 (define-key magit-svn-map (kbd "f") 'magit-svn-remote-update)
-(define-key magit-svn-map (kbd "s") 'magit-svn-find-rev)))
-(define-key magit-map (kbd "N") 'magit-svn-prefix)
+(define-key magit-svn-map (kbd "s") 'magit-svn-find-rev)
+(define-key magit-mode-map (kbd "N") 'magit-svn-prefix)
 
 (easy-menu-define magit-svn-extension-menu
   nil
@@ -171,9 +171,9 @@ If USE-CACHE is non nil, use the cached information."
     ["Commit" magit-svn-dcommit (magit-svn-enabled)]))
 (easy-menu-add-item 'magit-mode-menu '("Extensions") magit-svn-extension-menu)
 
-(add-hook magit-after-insert-unpulled-commits
+(add-hook 'magit-after-insert-unpulled-commits
           (lambda () (magit-insert-svn-unpulled t)))
-(add-hook magit-after-insert-unpushed-commits
+(add-hook 'magit-after-insert-unpushed-commits
           (lambda () (magit-insert-svn-unpushed t)))
 
 (add-hook 'magit-remote-string-hook 'magit-svn-remote-string)