<HTML><HEAD><TITLE>Samsung Enterprise Portal mySingle</TITLE> <META content="text/html; charset=euc-kr" http-equiv=Content-Type> <STYLE id=mysingle_style type=text/css>P { MARGIN-BOTTOM: 5px; FONT-SIZE: 9pt; FONT-FAMILY: 굴림체, arial; MARGIN-TOP: 5px } TD { MARGIN-BOTTOM: 5px; FONT-SIZE: 9pt; FONT-FAMILY: 굴림체, arial; MARGIN-TOP: 5px } LI { MARGIN-BOTTOM: 5px; FONT-SIZE: 9pt; FONT-FAMILY: 굴림체, arial; MARGIN-TOP: 5px } BODY { FONT-SIZE: 9pt; FONT-FAMILY: 굴림체, arial; MARGIN: 10px; LINE-HEIGHT: 1.4 } </STYLE> <META content=IE=5 http-equiv=X-UA-Compatible> <META content=IE=5 http-equiv=X-UA-Compatible> <META name=GENERATOR content="MSHTML 11.00.9600.18098"></HEAD> <BODY> I don't see any reason for the wl_display_add_socket_fd_auto and wl_display_add_socket_auto functions, as they are the same as passing NULL for the name to wl_display_add_socket_fd, right? ==> No, The wl_display_add_socket_fd_auto is different action to wl_display_add_socket_fd with NULL name. The wl_display_add_socket_fd_auto() is change display name(wayland-%d) until find success. <DIV dir=ltr>Also (on the patch) it looks like if there is an error it destroys the wl_socket object, could this do the wrong thing if the fd was supplied from outside? </DIV> <DIV class=gmail_extra> <DIV class=gmail_quote>On Thu, Nov 19, 2015 at 1:36 AM, Bryce Harrington <<A href="mailto:bryce@osg.samsung.com" target=_blank>bryce@osg.samsung.com</A>> wrote: <BLOCKQUOTE class=gmail_quote style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: #ccc 1px solid">Signed-off-by: Bryce Harrington <<A href="mailto:bryce@osg.samsung.com">bryce@osg.samsung.com</A>> --- src/wayland-server.c | 67 +++++++++++++++++++++++++++++++++++++++++----------- 1 file changed, 53 insertions(+), 14 deletions(-) diff --git a/src/wayland-server.c b/src/wayland-server.c index bac98bf..e84858d 100644 --- a/src/wayland-server.c +++ b/src/wayland-server.c @@ -1164,6 +1164,17 @@ _wl_display_add_socket(struct wl_display *display, struct wl_socket *s) return 0; } +/** Create a socket for clients with a specified file descriptor + * \param display Wayland display + * \param sock_fd The socket's file descriptor + * \return The socket name or NULL if failed. + * + * Adds a new socket with a specified file descriptor but an + * automatically determined socket name, in the form "wayland-N" + * where N is a display number. + * + * \memberof wl_display + */ WL_EXPORT const char * wl_display_add_socket_fd_auto(struct wl_display *display, int sock_fd) { @@ -1205,12 +1216,50 @@ wl_display_add_socket_fd_auto(struct wl_display *display, int sock_fd) return NULL; } +/** Create a socket for clients using default settings + * \param display The Wayland display to which the socket should be added + * \return The name of the created socket, or NULL if failed. + * + * This helper routine establishes a socket connection for Wayland clients + * using an automatically socket address and file descriptor. + * + * See wl_display_add_socket_fd_auto. + * + * \memberof wl_display + */ WL_EXPORT const char * wl_display_add_socket_auto(struct wl_display *display) { return wl_display_add_socket_fd_auto(display, -1); } +/** Add a socket with a defined file descriptor to the Wayland display for clients to connect. + * \param display Wayland display to which the socket should be added. + * \param name Name of the Unix socket. + * \param sock_fd File descriptor of the Unix socket. + * \return 0 if success. -1 if failed. + * + * This adds a Unix socket to Wayland display which can be used by clients to + * connect to Wayland display. + * + * If NULL is passed as name, then it would look for WAYLAND_DISPLAY env + * variable for the socket name. If WAYLAND_DISPLAY is not set, then default + * wayland-0 is used. + * + * If -1 is passed for sock_id, a new local socket will be created. A link to + * this socket will be made available in display->socket_list. + * + * This Unix socket will be created in the directory pointed to by environment + * variable XDG_RUNTIME_DIR. If XDG_RUNTIME_DIR is not set, then this function + * fails and returns -1. + * + * The length of socket path, i.e., the path set in XDG_RUNTIME_DIR and the + * socket name, must not exceed the maxium length of a Unix socket path. + * The function also fails if the user do not have write permission in the + * XDG_RUNTIME_DIR path or if the socket name is already in use. + * + * \memberof wl_display + */ WL_EXPORT int wl_display_add_socket_fd(struct wl_display *display, const char *name, int sock_fd) { @@ -1251,21 +1300,11 @@ wl_display_add_socket_fd(struct wl_display *display, const char *name, int sock_ * \param name Name of the Unix socket. * \return 0 if success. -1 if failed. * - * This adds a Unix socket to Wayland display which can be used by clients to - * connect to Wayland display. - * - * If NULL is passed as name, then it would look for WAYLAND_DISPLAY env - * variable for the socket name. If WAYLAND_DISPLAY is not set, then default - * wayland-0 is used. + * This adds a Unix socket to Wayland display with an automatically + * created file descriptor, which can be used by clients to connect to + * the Wayland display. * - * The Unix socket will be created in the directory pointed to by environment - * variable XDG_RUNTIME_DIR. If XDG_RUNTIME_DIR is not set, then this function - * fails and returns -1. - * - * The length of socket path, i.e., the path set in XDG_RUNTIME_DIR and the - * socket name, must not exceed the maxium length of a Unix socket path. - * The function also fails if the user do not have write permission in the - * XDG_RUNTIME_DIR path or if the socket name is already in use. + * See wl_display_add_socket_fd for more details. * * \memberof wl_display */ -- 1.9.1 _______________________________________________ wayland-devel mailing list <A href="mailto:wayland-devel@lists.freedesktop.org">wayland-devel@lists.freedesktop.org</A> <A href="http://lists.freedesktop.org/mailman/listinfo/wayland-devel" rel=noreferrer target=_blank>http://lists.freedesktop.org/mailman/listinfo/wayland-devel</A> </BLOCKQUOTE></DIV> </DIV> <TABLE id=confidentialsignimg> <TBODY> <TR> <TD NAMO_LOCK> <IMG border=0 src="cid:Z5JE7EUABGFC@namo.co.kr"></TD></TR></TBODY></TABLE></BODY></HTML><img src='http://ext.samsung.net/mailcheck/SeenTimeChecker?do=d1c4c6a24968fb25fbd3e29d1d5649074b93bddc36eef38cd261643f4e17952232c2d5b092db2212f255f74e40bd48632fd4ff3ea410d012fea75954dce1604da728c55b39cc59eacf878f9a26ce15a0' border=0 width=0 height=0 style='display:none'>