Author: gonzo
Date: Tue Jan 15 00:52:41 2019
New Revision: 343029
URL: https://svnweb.freebsd.org/changeset/base/343029

Log:
  [led] propagate error from set_led() to the caller
  
  Do not lose error condition by always returning 0 from set_led.
  None of the calls to set_led checks for return value at the moment so
  none of API consumers in base is affected.
  
  PR:           231567
  Submitted by: Bertrand Petit <bs...@phoe.frmug.org>
  MFC after:    1 week

Modified:
  head/sys/dev/led/led.c

Modified: head/sys/dev/led/led.c
==============================================================================
--- head/sys/dev/led/led.c      Tue Jan 15 00:37:37 2019        (r343028)
+++ head/sys/dev/led/led.c      Tue Jan 15 00:52:41 2019        (r343029)
@@ -261,7 +261,7 @@ led_set(char const *name, char const *cmd)
        mtx_unlock(&led_mtx);
        if (sb != NULL)
                sbuf_delete(sb);
-       return (0);
+       return (error);
 }
 
 static struct cdevsw led_cdevsw = {
_______________________________________________
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to