/mandos/release

To get this branch, use:
bzr branch http://bzr.recompile.se/loggerhead/mandos/release

« back to all changes in this revision

Viewing changes to mandos

  • Committer: Teddy Hogeborn
  • Date: 2013-10-28 20:08:32 UTC
  • mto: (237.7.272 trunk)
  • mto: This revision was merged to the branch mainline in revision 307.
  • Revision ID: teddy@recompile.se-20131028200832-80d1k8c5e8mugpkx
Syntax fix; use "raise" better in Mandos server.

* mandos: Syntax fix; use "raise" better in Mandos server; Always
          raise instances, not classes, skip argument when re-raising,
          etc.

Show diffs side-by-side

added added

removed removed

Lines of Context:
690
690
        # If a checker exists, make sure it is not a zombie
691
691
        try:
692
692
            pid, status = os.waitpid(self.checker.pid, os.WNOHANG)
693
 
        except (AttributeError, OSError) as error:
694
 
            if (isinstance(error, OSError)
695
 
                and error.errno != errno.ECHILD):
696
 
                raise error
 
693
        except AttributeError:
 
694
            pass
 
695
        except OSError as error:
 
696
            if error.errno != errno.ECHILD:
 
697
                raise
697
698
        else:
698
699
            if pid:
699
700
                logger.warning("Checker was a zombie")
1702
1703
            logger.debug("Protocol version: %r", line)
1703
1704
            try:
1704
1705
                if int(line.strip().split()[0]) > 1:
1705
 
                    raise RuntimeError
 
1706
                    raise RuntimeError(line)
1706
1707
            except (ValueError, IndexError, RuntimeError) as error:
1707
1708
                logger.error("Unknown protocol version: %s", error)
1708
1709
                return
1915
1916
    
1916
1917
    def add_pipe(self, parent_pipe, proc):
1917
1918
        """Dummy function; override as necessary"""
1918
 
        raise NotImplementedError
 
1919
        raise NotImplementedError()
1919
1920
 
1920
1921
 
1921
1922
class IPv6_TCPServer(MultiprocessingMixInWithPipe,
2258
2259
            else:
2259
2260
                raise ValueError("Unknown suffix {0!r}"
2260
2261
                                 .format(suffix))
2261
 
        except (ValueError, IndexError) as e:
 
2262
        except IndexError as e:
2262
2263
            raise ValueError(*(e.args))
2263
2264
        timevalue += delta
2264
2265
    return timevalue
2481
2482
        os.setuid(uid)
2482
2483
    except OSError as error:
2483
2484
        if error.errno != errno.EPERM:
2484
 
            raise error
 
2485
            raise
2485
2486
    
2486
2487
    if debug:
2487
2488
        # Enable all possible GnuTLS debugging
2792
2793
            else:
2793
2794
                logger.warning("Could not save persistent state:",
2794
2795
                               exc_info=e)
2795
 
                raise e
 
2796
                raise
2796
2797
        
2797
2798
        # Delete all clients, and settings from config
2798
2799
        while tcp_server.clients: