Re: [PATCH 2/4] idmapd: Add graceful exit and resource cleanup

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2020-07-23 13:56, Steve Dickson wrote:
@@ -290,6 +306,9 @@ main(int argc, char **argv)
  			serverstart = 0;
  	}
+ /* Not needed anymore */
+	conf_cleanup();
+
I'm a bit confused by this comment... If it is not needed why as the call added?

Sorry, I should have been a bit more verbose in the comment. I meant that we didn't need access to the config file anymore (we've already looked up everything) and can free those resources early.

Perhaps /* Config not needed anymore */ or something.

Doug




[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux