1. 24 3月, 2008 2 次提交
  2. 25 2月, 2008 1 次提交
    • J
      Avoid new "make syntax-check" failures. · e497deb8
      Jim Meyering 提交于
      * .x-sc_avoid_write: Exempt src/util-lib.c, too.
      * Makefile.maint (sc_unmarked_diagnostics): Filter out false positives.
      * src/storage_backend_fs.c (virStorageBackendFileSystemVolCreate):
      Use safewrite, not write.
      * src/storage_backend_logical.c (virStorageBackendLogicalBuildPool): Likewise.
      e497deb8
  3. 22 2月, 2008 1 次提交
    • J
      Use safewrite in place of write, in many cases. · b59d9c85
      Jim Meyering 提交于
      Also add "make syntax-check" rules to ensure no new uses sneak in.
      
      There are many uses of write like this:
      
          if (write (fd, xml, towrite) != towrite)
              return -1;
      
      The problem is that the syscall can succeed, yet write less than
      the requested number of bytes, so the caller should retry
      rather than simply failing.
      
      This patch changes most of them to use util.c's safewrite wrapper,
      which encapsulates the process.  Also, there were a few cases in
      which the retry loop was open-coded, and I replaced those, too.
      
      * Makefile.maint (sc_avoid_write): New rule, to avoid recurrence.
      * .x-sc_avoid_write: New file.  Record two legitimate exemptions.
      * qemud/qemud.c (sig_handler, qemudClientWriteBuf): Use safewrite, not write.
      * src/conf.c (__virConfWriteFile): Likewise.
      * src/qemu_conf.c (qemudSaveConfig, qemudSaveNetworkConfig): Likewise.
      * src/qemu_driver.c (qemudWaitForMonitor, qemudStartVMDaemon)
      (qemudVMData, PROC_IP_FORWARD): Likewise.
      * proxy/libvirt_proxy.c: Include "util.h".
      (proxyWriteClientSocket): Use safewrite.
      * src/test.c (testDomainSave, testDomainCoreDump): Likewise.
      * src/proxy_internal.c (virProxyWriteClientSocket): Likewise.
      * src/virsh.c: Include "util-lib.h".
      (vshOutputLogFile): Use safewrite.
      * src/console.c: Include "util-lib.h".
      (vshRunConsole): Use safewrite.
      b59d9c85
  4. 20 2月, 2008 1 次提交
  5. 08 2月, 2008 3 次提交
    • J
      Enable another syntax-check rule. · db4037e5
      Jim Meyering 提交于
      * Makefile.cfg (local-checks-to-skip): Remove sc_no_have_config_h.
      * qemud/mdns.c: Remove "#ifdef HAVE_CONFIG_H".
      * Makefile.maint (sc_no_have_config_h): Tighten up regexp,
      so that the above mention of "HAVE_CONFIG_H" doesn't match.
      * .x-sc_no_have_config_h: New file, to exempt gnulib/ straggler.
      db4037e5
    • J
      Mark all qemudLog diagnostics for translation. · 247df6e4
      Jim Meyering 提交于
      * po/POTFILES.in: Add names of many new files.
      * Makefile.maint (err_func_re): Add qemudLog.
      Mark diagnostics with _(...).  Split some long lines.
      * qemud/qemud.c (remoteCheckCertFile, remoteInitializeGnuTLS):
      (qemudDispatchSignalEvent, qemudSetCloseExec, qemudSetNonBlock):
      (qemudWritePidFile, qemudListenUnix, remoteMakeSockets):
      (remoteListenTCP, qemudInitPaths, qemudInitialize):
      (qemudNetworkInit, remoteInitializeTLSSession, remoteCheckDN):
      (remoteCheckCertificate, remoteCheckAccess, qemudDispatchServer):
      (qemudClientReadBuf, qemudDispatchClientRead):
      (qemudClientWriteBuf, qemudDispatchClientWrite, qemudOneLoop):
      (remoteConfigGetStringList, checkType, GET_CONF_STR):
      (remoteConfigGetAuth, remoteReadConfigFile, main):
      * qemud/remote.c (remoteDispatchAuthSaslInit, remoteSASLCheckSSF):
      (remoteSASLCheckAccess, remoteDispatchAuthSaslStart):
      (remoteDispatchAuthSaslStep, remoteDispatchAuthSaslInit):
      (remoteDispatchAuthSaslStart, remoteDispatchAuthSaslStep):
      (qemudGetSocketIdentity, remoteDispatchAuthPolkit):
      * src/iptables.c (notifyRulesUpdated, MAX_FILE_LEN, iptRulesSave):
      (iptRulesReload):
      * src/qemu_conf.c (qemudExtractVersionInfo, qemudLoadConfig):
      (qemudLoadNetworkConfig, qemudScanConfigDir):
      * src/qemu_driver.c (qemudSetCloseExec, qemudSetNonBlock):
      (qemudAutostartConfigs, qemudStartup, qemudReload):
      (qemudWaitForMonitor, qemudStartVMDaemon, qemudVMData):
      (qemudShutdownVMDaemon, qemudStartNetworkDaemon):
      (qemudShutdownNetworkDaemon, qemudMonitorCommand):
      (qemudDomainUndefine, qemudNetworkUndefine):
      * src/uuid.c (virUUIDGenerate):
      * src/xm_internal.c (xenXMAttachInterface):
      247df6e4
    • J
      Remove more useless if tests before "free"-like functions. · e8ff93b4
      Jim Meyering 提交于
      * build-aux/useless-if-before-free: Rename from ...
      * build-aux/find-unnecessary-if-before-free: ... this.  Remove file.
      Also changed it so that new names are no longer hard-coded in the
      script.  Instead, they're supplied via options:
      * Makefile.cfg (useless_free_options): Define.
      Add xmlXPathFreeObject to the list of free-like functions it detects.
      * Makefile.maint (sc_avoid_if_before_free): Reflect script renaming.
      * .x-sc_avoid_if_before_free: Likewise.
      * src/openvz_conf.c (openvzParseXML): Remove useless "if"-before-free.
      * src/qemu_conf.c (qemudParseXML, qemudParseNetworkXML): Likewise.
      * src/virsh.c (cmdVNCDisplay, cmdTTYConsole, cmdDetachInterface):
      (cmdDetachDisk): Likewise.
      * src/xm_internal.c (xenXMConfigSetIntFromXPath): Likewise.
      (xenXMConfigSetStringFromXPath, xenXMParseXMLToConfig): Likewise.
      (xenXMDomainAttachDevice, xenXMAttachDisk, xenXMAttachInterface):
      (xenXMDomainDetachDevice): Likewise.
      * src/xml.c (virXPathString): Likewise.
      * tests/xmlrpctest.c (checkRequestValue): Likewise.
      e8ff93b4
  6. 05 2月, 2008 2 次提交
    • J
      Eliminate all uses of virBufferAdd with string literals. · fb66d5ab
      Jim Meyering 提交于
      * Makefile.maint (sc_prohibit_virBufferAdd_with_string_literal):
      New rule.
      * src/buf.h (virBufferAddLit): Define.
      * src/conf.c (virConfSaveValue): Use virBufferAddLit, in place
      of virBufferAdd everywhere possible.
      (virConfSaveEntry): Likewise.
      * src/qemu_conf.c (qemudGenerateXML, qemudGenerateNetworkXML): Likewise.
      * src/qemu_driver.c (qemudGetFeatures, qemudGetCapabilities): Likewise.
      * src/test.c (testDomainDumpXML, testNetworkDumpXML): Likewise.
      * src/xen_internal.c (xenHypervisorMakeCapabilitiesXML): Likewise.
      * src/xend_internal.c (xend_parse_sexp_desc_os): Likewise.
      (xend_parse_sexp_desc, sexpr_to_xend_topology_xml): Likewise.
      * src/xm_internal.c (xenXMDomainFormatXML, xenXMDomainPinVcpu): Likewise.
      * src/xml.c (virSaveCpuSet, virParseXenCpuTopology): Likewise.
      (virDomainParseXMLGraphicsDescImage): Likewise.
      (virDomainParseXMLGraphicsDescVFB, virDomainParseXMLOSDescHVM): Likewise.
      (virDomainParseXMLOSDescPV, virDomainParseXMLDiskDesc): Likewise.
      (virDomainParseXMLIfDesc, virDomainParseXMLDesc): Likewise.
      fb66d5ab
    • J
      Enable more checks for unmarked messages; fix violators. · ecaa6962
      Jim Meyering 提交于
      * Makefile.maint (sc_unmarked_diagnostics): Check for more functions.
      * src/conf.c (virConfParseValue): Remove trailing space in diagnostic.
      * src/libvirt.c (do_open, virDomainMigrate): Mark diagnostics.
      * src/xend_internal.c (do_connect, xenDaemonDomainDumpXMLByID): Likewise.
      (xenDaemonDomainDumpXMLByName, xenDaemonCreateLinux): Likewise.
      (xenDaemonDomainMigratePerform, xenDaemonDomainDefineXML): Likewise.
      * src/xml.c (virXPathString, virXPathNumber, virXPathLong): Likewise.
      (virXPathBoolean, virXPathNode, virXPathNodeSet): Likewise.
      (virDomainParseXMLOSDescHVM): Likewise.
      * src/xmlrpc.c (xmlRpcValueNew): Likewise.
      (xmlRpcValueUnmarshalDictElement): Likewise.
      ecaa6962
  7. 30 1月, 2008 1 次提交
    • J
      Add framework for code style- and syntax-checking rules. · cc337da2
      Jim Meyering 提交于
      	Almost all tests are initially disabled via the list in Makefile.cfg.
      	* Makefile.am (EXTRA_DIST): Add .x-sc_avoid_if_before_free.
      	Omit names of files that automake includes automatically.
      	* .x-sc_avoid_if_before_free: New file.
      	* build-aux/vc-list-files: Likewise.
      	* build-aux/find-unnecessary-if-before-free: Likewise.
      	* GNUmakefile, Makefile.cfg, Makefile.maint: New files.
      cc337da2