Sync to Ic3303be0c22063abab4684119bd088809838c522

    Add accessibility events & Change 24 -> 32

    Pixmap is created as 32 bits pixels. (RGBA)
    So every client & provider library should access it with 32 bits mode.

    This patch changes 24 bits to 32 bits.

    Add hold_scroll and remove unused string in packet.

    Provider needs to send hold scroll request to viewer.
    In case of scrolling pages from the viewer with mouse_event=true boxes.
    user can confusing with event processing.

    While scrolling the pages, the boxes are able to try to get mouse event from user.
    Then the event and its action can be conflicted.

    To resolve this, the provider should send the scroll hold request to the viewer.
    If viewer get that event, has to freeze the scroller to prevent from event confliction.

Change-Id: I0097f1e67aac5dd3726d753f184a4fc022049a5b

diff --git a/include/provider.h b/include/provider.h
index 15a06a6..3e591cc 100644 (file)
--- a/include/provider.h
+++ b/include/provider.h
@@ -40,12 +40,12 @@ struct event_info {
 };
 
 enum access_event {
-       ACCESS_READ,
-       ACCESS_READ_NEXT,
-       ACCESS_READ_PREV,
+       ACCESS_HIGHLIGHT,
+       ACCESS_HIGHLIGHT_NEXT,
+       ACCESS_HIGHLIGHT_PREV,
        ACCESS_ACTIVATE,
-       ACCESS_UP,
-       ACCESS_DOWN,
+       ACCESS_VALUE_CHANGE,
+       ACCESS_SCROLL,
 };
 
 struct event_arg {
@@ -263,7 +263,7 @@ struct event_handler {
  * \param[in] disp XDisplay object, if you don't know what this is, set NULL
  * \param[in] name Slave name which is given by the master provider.
  * \param[in] table Event handler table
- * \return int Success 0 otherwise errno < 0
+ * \return int Success LB_STATUS_SUCCESS otherwise errno < 0
  */
 extern int provider_init(void *disp, const char *name, struct event_handler *table, void *data);
 
@@ -276,13 +276,13 @@ extern void *provider_fini(void);
  * \brief Send the hello signal to the master
  *        Master will activate connection of this only if you send this hello event.
  *        or the master will reject all requests from your provider.
- * \return int Success 0 otherwise errno < 0
+ * \return int Success LB_STATUS_SUCCESS otherwise errno < 0
  */
 extern int provider_send_hello(void);
 
 /*!
  * \brief Send the ping message to the master to notify that your provider is working properly.
- * \return int Success 0 otherwise errno < 0
+ * \return int Success LB_STATUS_SUCCESS otherwise errno < 0
  */
 extern int provider_send_ping(void);
 
@@ -293,7 +293,7 @@ extern int provider_send_ping(void);
  * \param[in] w Width of an updated content
  * \param[in] h Height of an updated content
  * \param[in] priority Priority of an updated content
- * \return int Success 0 otherwise errno < 0
+ * \return int Success LB_STATUS_SUCCESS otherwise errno < 0
  */
 extern int provider_send_updated(const char *pkgname, const char *id, int w, int h, double priority, const char *content, const char *title);
 
@@ -302,7 +302,7 @@ extern int provider_send_updated(const char *pkgname, const char *id, int w, int
  * \param[in] pkgname Package name of an updated livebox.
  * \param[in] id Instance ID of an updated livebox.
  * \param[in] descfile The filename of a description file.
- * \return int Success 0 otherwise errno < 0
+ * \return int Success LB_STATUS_SUCCESS otherwise errno < 0
  */
 extern int provider_send_desc_updated(const char *pkgname, const char *id, const char *descfile);
 
@@ -310,7 +310,7 @@ extern int provider_send_desc_updated(const char *pkgname, const char *id, const
  * \brief Send the deleted event of specified livebox instance
  * \param[in] pkgname Package name of the livebox
  * \param[in] id Livebox instance ID
- * \return int Success 0 otherwise errno < 0
+ * \return int Success LB_STATUS_SUCCESS otherwise errno < 0
  */
 extern int provider_send_deleted(const char *pkgname, const char *id);
 
@@ -320,7 +320,7 @@ extern int provider_send_deleted(const char *pkgname, const char *id);
  * \param[in] pkgname Package name of the livebox
  * \param[in] id Instance ID of the livebox
  * \param[in] funcname Function name which will be called
- * \return int Success 0 otherwise errno < 0
+ * \return int Success LB_STATUS_SUCCESS otherwise errno < 0
  */
 extern int provider_send_ret(const char *pkgname, const char *id, const char *funcname);
 
@@ -330,7 +330,7 @@ extern int provider_send_ret(const char *pkgname, const char *id, const char *fu
  * \param[in] pkgname Package name of the livebox
  * \param[in] id Instance ID of the livebox
  * \param[in] funcname Function name which is called by the slave
- * \return int Success 0 otherwise errno < 0
+ * \return int Success LB_STATUS_SUCCESS otherwise errno < 0
  */
 extern int provider_send_call(const char *pkgname, const char *id, const char *funcname);
 
@@ -340,10 +340,20 @@ extern int provider_send_call(const char *pkgname, const char *id, const char *f
  * \param[in] pkgname Package name of the livebox
  * \param[in] id ID of the livebox instance
  * \param[in] funcname Reason of the fault error
- * \return int Success 0 otherwise errno < 0
+ * \return int Success LB_STATUS_SUCCESS otherwise errno < 0
  */
 extern int provider_send_faulted(const char *pkgname, const char *id, const char *funcname);
 
+/*!
+ * \brief If you want notify to viewer to seize the screen,
+ *        prevent moving a box from user event
+ * \param[in] pkgname Package name of the livebox
+ * \param[in] id ID of the livebox instance
+ * \param[in] seize 1 if viewer needs to hold a box, or 0
+ * \return int Success LB_STATUS_SUCCESS othere LB_STATUS_ERROR_XXX
+ */
+extern int provider_send_hold_scroll(const char *pkgname, const char *id, int seize);
+
 #ifdef __cplusplus
 }
 #endif

diff --git a/packaging/libprovider.spec b/packaging/libprovider.spec
index da03a04..dde7826 100644 (file)
--- a/packaging/libprovider.spec
+++ b/packaging/libprovider.spec
@@ -1,6 +1,6 @@
 Name: libprovider
 Summary: Library for developing the livebox service provider.
-Version: 0.6.2
+Version: 0.7.0
 Release: 1
 Group: HomeTF/Livebox
 License: Flora License

diff --git a/src/fb.c b/src/fb.c
index ff850d2..4e649c2 100644 (file)
--- a/src/fb.c
+++ b/src/fb.c
@@ -435,7 +435,7 @@ static inline int create_pixmap_info(struct fb_info *info)
         * \NOTE
         * XCreatePixmap can only uses 24 bits depth only.
         */
-       pixmap_info->xim = XShmCreateImage(s_info.disp, s_info.visual, 24/* (s_info.depth << 3) */, ZPixmap, NULL, &pixmap_info->si, info->w, info->h);
+       pixmap_info->xim = XShmCreateImage(s_info.disp, s_info.visual, (s_info.depth << 3), ZPixmap, NULL, &pixmap_info->si, info->w, info->h);
        if (pixmap_info->xim == NULL) {
                if (shmdt(pixmap_info->si.shmaddr) < 0)
                        ErrPrint("shmdt: %s\n", strerror(errno));

diff --git a/src/provider.c b/src/provider.c
index 475071c..70b4f7c 100644 (file)
--- a/src/provider.c
+++ b/src/provider.c
@@ -563,23 +563,64 @@ out:
        return NULL;
 }
 
-struct packet *master_pd_access_read(pid_t pid, int handle, const struct packet *packet)
+struct packet *master_pd_access_value_change(pid_t pid, int handle, const struct packet *packet)
 {
        struct event_arg arg;
        double timestamp;
        int ret;
 
-       ret = packet_get(packet, "ssiiddd", &arg.pkgname, &arg.id,
-                                        &arg.info.pd_access.w, &arg.info.pd_access.h,
+       ret = packet_get(packet, "ssdii", &arg.pkgname, &arg.id, &timestamp, &arg.info.pd_access.x, &arg.info.pd_access.y);
+       if (ret != 5) {
+               ErrPrint("Invalid packet\n");
+               goto out;
+       }
+
+       arg.type = EVENT_PD_ACCESS;
+       arg.info.pd_access.event = ACCESS_VALUE_CHANGE;
+       if (s_info.table.pd_access)
+               (void)s_info.table.pd_access(&arg, s_info.data);
+
+out:
+       return NULL;
+}
+
+struct packet *master_pd_access_scroll(pid_t pid, int handle, const struct packet *packet)
+{
+       struct event_arg arg;
+       double timestamp;
+       int ret;
+
+       ret = packet_get(packet, "ssdii", &arg.pkgname, &arg.id, &timestamp, &arg.info.pd_access.x, &arg.info.pd_access.y);
+       if (ret != 5) {
+               ErrPrint("Invalid packet\n");
+               goto out;
+       }
+
+       arg.type = EVENT_PD_ACCESS;
+       arg.info.pd_access.event = ACCESS_SCROLL;
+       if (s_info.table.pd_access)
+               (void)s_info.table.pd_access(&arg, s_info.data);
+
+out:
+       return NULL;
+}
+
+struct packet *master_pd_access_hl(pid_t pid, int handle, const struct packet *packet)
+{
+       struct event_arg arg;
+       double timestamp;
+       int ret;
+
+       ret = packet_get(packet, "ssdii", &arg.pkgname, &arg.id,
                                         &timestamp,
                                         &arg.info.pd_access.x, &arg.info.pd_access.y);
-       if (ret != 7) {
+       if (ret != 5) {
                ErrPrint("Invalid packet\n");
                goto out;
        }
 
        arg.type = EVENT_PD_ACCESS;
-       arg.info.pd_access.event = ACCESS_READ;
+       arg.info.pd_access.event = ACCESS_HIGHLIGHT;
        if (s_info.table.pd_access)
                (void)s_info.table.pd_access(&arg, s_info.data);
 
@@ -587,23 +628,22 @@ out:
        return NULL;
 }
 
-struct packet *master_pd_access_read_prev(pid_t pid, int handle, const struct packet *packet)
+struct packet *master_pd_access_hl_prev(pid_t pid, int handle, const struct packet *packet)
 {
        struct event_arg arg;
        double timestamp;
        int ret;
 
-       ret = packet_get(packet, "ssiiddd", &arg.pkgname, &arg.id,
-                                        &arg.info.pd_access.w, &arg.info.pd_access.h,
+       ret = packet_get(packet, "ssdii", &arg.pkgname, &arg.id,
                                         &timestamp,
                                         &arg.info.pd_access.x, &arg.info.pd_access.y);
-       if (ret != 7) {
+       if (ret != 5) {
                ErrPrint("Invalid packet\n");
                goto out;
        }
 
        arg.type = EVENT_PD_ACCESS;
-       arg.info.pd_access.event = ACCESS_READ_PREV;
+       arg.info.pd_access.event = ACCESS_HIGHLIGHT_PREV;
        if (s_info.table.pd_access)
                (void)s_info.table.pd_access(&arg, s_info.data);
 
@@ -611,23 +651,22 @@ out:
        return NULL;
 }
 
-struct packet *master_pd_access_read_next(pid_t pid, int handle, const struct packet *packet)
+struct packet *master_pd_access_hl_next(pid_t pid, int handle, const struct packet *packet)
 {
        struct event_arg arg;
        double timestamp;
        int ret;
 
-       ret = packet_get(packet, "ssiiddd", &arg.pkgname, &arg.id,
-                                        &arg.info.pd_access.w, &arg.info.pd_access.h,
+       ret = packet_get(packet, "ssdii", &arg.pkgname, &arg.id,
                                         &timestamp,
                                         &arg.info.pd_access.x, &arg.info.pd_access.y);
-       if (ret != 7) {
+       if (ret != 5) {
                ErrPrint("Invalid packet\n");
                goto out;
        }
 
        arg.type = EVENT_PD_ACCESS;
-       arg.info.pd_access.event = ACCESS_READ_NEXT;
+       arg.info.pd_access.event = ACCESS_HIGHLIGHT_NEXT;
        if (s_info.table.pd_access)
                (void)s_info.table.pd_access(&arg, s_info.data);
 
@@ -641,11 +680,10 @@ struct packet *master_pd_access_activate(pid_t pid, int handle, const struct pac
        double timestamp;
        int ret;
 
-       ret = packet_get(packet, "ssiiddd", &arg.pkgname, &arg.id,
-                                        &arg.info.pd_access.w, &arg.info.pd_access.h,
+       ret = packet_get(packet, "ssdii", &arg.pkgname, &arg.id,
                                         &timestamp,
                                         &arg.info.pd_access.x, &arg.info.pd_access.y);
-       if (ret != 7) {
+       if (ret != 5) {
                ErrPrint("Invalid packet\n");
                goto out;
        }
@@ -659,23 +697,22 @@ out:
        return NULL;
 }
 
-struct packet *master_lb_access_read(pid_t pid, int handle, const struct packet *packet)
+struct packet *master_lb_access_hl(pid_t pid, int handle, const struct packet *packet)
 {
        struct event_arg arg;
        double timestamp;
        int ret;
 
-       ret = packet_get(packet, "ssiiddd", &arg.pkgname, &arg.id,
-                                        &arg.info.lb_access.w, &arg.info.lb_access.h,
+       ret = packet_get(packet, "ssdii", &arg.pkgname, &arg.id,
                                         &timestamp,
                                         &arg.info.lb_access.x, &arg.info.lb_access.y);
-       if (ret != 7) {
+       if (ret != 5) {
                ErrPrint("Invalid packet\n");
                goto out;
        }
 
        arg.type = EVENT_LB_ACCESS;
-       arg.info.lb_access.event = ACCESS_READ;
+       arg.info.lb_access.event = ACCESS_HIGHLIGHT;
        if (s_info.table.lb_access)
                (void)s_info.table.lb_access(&arg, s_info.data);
 
@@ -683,23 +720,22 @@ out:
        return NULL;
 }
 
-struct packet *master_lb_access_read_prev(pid_t pid, int handle, const struct packet *packet)
+struct packet *master_lb_access_hl_prev(pid_t pid, int handle, const struct packet *packet)
 {
        struct event_arg arg;
        double timestamp;
        int ret;
 
-       ret = packet_get(packet, "ssiiddd", &arg.pkgname, &arg.id,
-                                        &arg.info.lb_access.w, &arg.info.lb_access.h,
+       ret = packet_get(packet, "ssdii", &arg.pkgname, &arg.id,
                                         &timestamp,
                                         &arg.info.lb_access.x, &arg.info.lb_access.y);
-       if (ret != 7) {
+       if (ret != 5) {
                ErrPrint("Invalid packet\n");
                goto out;
        }
 
        arg.type = EVENT_LB_ACCESS;
-       arg.info.lb_access.event = ACCESS_READ_PREV;
+       arg.info.lb_access.event = ACCESS_HIGHLIGHT_PREV;
        if (s_info.table.lb_access)
                (void)s_info.table.lb_access(&arg, s_info.data);
 
@@ -707,23 +743,68 @@ out:
        return NULL;
 }
 
-struct packet *master_lb_access_read_next(pid_t pid, int handle, const struct packet *packet)
+struct packet *master_lb_access_hl_next(pid_t pid, int handle, const struct packet *packet)
 {
        struct event_arg arg;
        double timestamp;
        int ret;
 
-       ret = packet_get(packet, "ssiiddd", &arg.pkgname, &arg.id,
-                                        &arg.info.lb_access.w, &arg.info.lb_access.h,
+       ret = packet_get(packet, "ssdii", &arg.pkgname, &arg.id,
                                         &timestamp,
                                         &arg.info.lb_access.x, &arg.info.lb_access.y);
-       if (ret != 7) {
+       if (ret != 5) {
                ErrPrint("Invalid packet\n");
                goto out;
        }
 
        arg.type = EVENT_LB_ACCESS;
-       arg.info.lb_access.event = ACCESS_READ_NEXT;
+       arg.info.lb_access.event = ACCESS_HIGHLIGHT_NEXT;
+       if (s_info.table.lb_access)
+               (void)s_info.table.lb_access(&arg, s_info.data);
+
+out:
+       return NULL;
+}
+
+struct packet *master_lb_access_value_change(pid_t pid, int handle, const struct packet *packet)
+{
+       struct event_arg arg;
+       double timestamp;
+       int ret;
+
+       ret = packet_get(packet, "ssdii", &arg.pkgname, &arg.id,
+                                        &timestamp,
+                                        &arg.info.lb_access.x, &arg.info.lb_access.y);
+       if (ret != 5) {
+               ErrPrint("Invalid packet\n");
+               goto out;
+       }
+
+       arg.type = EVENT_LB_ACCESS;
+       arg.info.lb_access.event = ACCESS_VALUE_CHANGE;
+       if (s_info.table.lb_access)
+               (void)s_info.table.lb_access(&arg, s_info.data);
+
+out:
+       return NULL;
+}
+
+struct packet *master_lb_access_scroll(pid_t pid, int handle, const struct packet *packet)
+{
+       struct event_arg arg;
+       double timestamp;
+       int ret;
+
+       ret = packet_get(packet, "ssdii", &arg.pkgname, &arg.id,
+                                        &timestamp,
+                                        &arg.info.lb_access.x, &arg.info.lb_access.y);
+       if (ret != 5) {
+               ErrPrint("Invalid packet\n");
+               goto out;
+       }
+
+       arg.type = EVENT_LB_ACCESS;
+       arg.info.lb_access.event = ACCESS_SCROLL;
        if (s_info.table.lb_access)
                (void)s_info.table.lb_access(&arg, s_info.data);
 
@@ -737,11 +818,10 @@ struct packet *master_lb_access_activate(pid_t pid, int handle, const struct pac
        double timestamp;
        int ret;
 
-       ret = packet_get(packet, "ssiiddd", &arg.pkgname, &arg.id,
-                                        &arg.info.lb_access.w, &arg.info.lb_access.h,
+       ret = packet_get(packet, "ssdii", &arg.pkgname, &arg.id,
                                         &timestamp,
                                         &arg.info.lb_access.x, &arg.info.lb_access.y);
-       if (ret != 7) {
+       if (ret != 5) {
                ErrPrint("Invalid packet\n");
                goto out;
        }
@@ -885,39 +965,58 @@ static struct method s_table[] = {
                .cmd = "resume",
                .handler = master_resume, /* timestamp, ret */
        },
+
        {
-               .cmd = "pd_access_read",
-               .handler = master_pd_access_read,
+               .cmd = "pd_access_hl",
+               .handler = master_pd_access_hl,
        },
        {
-               .cmd = "pd_access_read_prev",
-               .handler = master_pd_access_read_prev,
+               .cmd = "pd_access_hl_prev",
+               .handler = master_pd_access_hl_prev,
        },
        {
-               .cmd = "pd_access_read_next",
-               .handler = master_pd_access_read_next,
+               .cmd = "pd_access_hl_next",
+               .handler = master_pd_access_hl_next,
        },
        {
                .cmd = "pd_access_activate",
                .handler = master_pd_access_activate,
        },
        {
-               .cmd = "lb_access_read",
-               .handler = master_lb_access_read,
+               .cmd = "pd_access_scroll",
+               .handler = master_pd_access_scroll,
        },
        {
-               .cmd = "lb_access_read_prev",
-               .handler = master_lb_access_read_prev,
+               .cmd = "pd_access_value_change",
+               .handler = master_pd_access_value_change,
        },
+
        {
-               .cmd = "lb_access_read_next",
-               .handler = master_lb_access_read_next,
+               .cmd = "lb_access_hl",
+               .handler = master_lb_access_hl,
+       },
+       {
+               .cmd = "lb_access_hl_prev",
+               .handler = master_lb_access_hl_prev,
+       },
+       {
+               .cmd = "lb_access_hl_next",
+               .handler = master_lb_access_hl_next,
        },
        {
                .cmd = "lb_access_activate",
                .handler = master_lb_access_activate,
        },
        {
+               .cmd = "lb_access_scroll",
+               .handler = master_lb_access_scroll,
+       },
+       {
+               .cmd = "lb_access_value_change",
+               .handler = master_lb_access_value_change,
+       },
+
+       {
                .cmd = NULL,
                .handler = NULL,
        },
@@ -1040,7 +1139,7 @@ EAPI int provider_send_call(const char *pkgname, const char *id, const char *fun
                return LB_STATUS_ERROR_INVALID;
        }
 
-       packet = packet_create_noack("call", "ssss", s_info.name, pkgname, id, funcname);
+       packet = packet_create_noack("call", "sss", pkgname, id, funcname);
        if (!packet) {
                ErrPrint("Failed to create a packet\n");
                return LB_STATUS_ERROR_FAULT;
@@ -1061,7 +1160,7 @@ EAPI int provider_send_ret(const char *pkgname, const char *id, const char *func
                return LB_STATUS_ERROR_INVALID;
        }
 
-       packet = packet_create_noack("ret", "ssss", s_info.name, pkgname, id, funcname);
+       packet = packet_create_noack("ret", "sss", pkgname, id, funcname);
        if (!packet) {
                ErrPrint("Failed to create a packet\n");
                return LB_STATUS_ERROR_FAULT;
@@ -1082,7 +1181,7 @@ EAPI int provider_send_faulted(const char *pkgname, const char *id, const char *
                return LB_STATUS_ERROR_INVALID;
        }
 
-       packet = packet_create_noack("faulted", "ssss", s_info.name, pkgname, id, funcname);
+       packet = packet_create_noack("faulted", "sss", pkgname, id, funcname);
        if (!packet) {
                ErrPrint("Failed to create a packet\n");
                return LB_STATUS_ERROR_FAULT;
@@ -1218,8 +1317,8 @@ EAPI int provider_send_updated(const char *pkgname, const char *id, int w, int h
 
        keep_file_in_safe(id);
 
-       packet = packet_create_noack("updated", "sssiidss",
-                                       s_info.name, pkgname, id, w, h, priority, content_info, title);
+       packet = packet_create_noack("updated", "ssiidss",
+                                       pkgname, id, w, h, priority, content_info, title);
        if (!packet) {
                ErrPrint("failed to build a packet\n");
                return LB_STATUS_ERROR_FAULT;
@@ -1248,7 +1347,7 @@ EAPI int provider_send_desc_updated(const char *pkgname, const char *id, const c
        if (!descfile)
                descfile = util_uri_to_path(id); /* In case of the NULL descfilename, use the ID */
 
-       packet = packet_create_noack("desc_updated", "ssss", s_info.name, pkgname, id, descfile);
+       packet = packet_create_noack("desc_updated", "sss", pkgname, id, descfile);
        if (!packet) {
                ErrPrint("Failed to build a packet\n");
                return LB_STATUS_ERROR_FAULT;
@@ -1274,7 +1373,33 @@ EAPI int provider_send_deleted(const char *pkgname, const char *id)
                return LB_STATUS_ERROR_INVALID;
        }
 
-       packet = packet_create_noack("deleted", "sss", s_info.name, pkgname, id);
+       packet = packet_create_noack("deleted", "ss", pkgname, id);
+       if (!packet) {
+               ErrPrint("Failed to build a packet\n");
+               return LB_STATUS_ERROR_FAULT;
+       }
+
+       ret = com_core_packet_send_only(s_info.fd, packet);
+       packet_destroy(packet);
+       return ret < 0 ? LB_STATUS_ERROR_FAULT : LB_STATUS_SUCCESS;
+}
+
+EAPI int provider_send_hold_scroll(const char *pkgname, const char *id, int hold)
+{
+       struct packet *packet;
+       int ret;
+
+       if (!pkgname || !id) {
+               ErrPrint("Invalid argument\n");
+               return LB_STATUS_ERROR_INVALID;
+       }
+
+       if (s_info.fd < 0) {
+               ErrPrint("Connection is not established\n");
+               return LB_STATUS_ERROR_INVALID;
+       }
+
+       packet = packet_create_noack("scroll", "ssi", pkgname, id, !!hold);
        if (!packet) {
                ErrPrint("Failed to build a packet\n");
                return LB_STATUS_ERROR_FAULT;

diff --git a/src/provider_buffer.c b/src/provider_buffer.c
index d157023..c61f061 100644 (file)
--- a/src/provider_buffer.c
+++ b/src/provider_buffer.c
@@ -53,7 +53,7 @@ static inline struct fb_info *send_acquire_request(enum target_type type, const
        struct fb_info *ret;
        int status;
 
-       packet = packet_create("acquire_buffer", "isssiii", type, provider_name(), pkgname, id, w, h, size);
+       packet = packet_create("acquire_buffer", "issiii", type, pkgname, id, w, h, size);
        if (!packet) {
                ErrPrint("Failed to build a packet\n");
                return NULL;
@@ -97,7 +97,7 @@ static inline int send_release_request(enum target_type type, const char *pkgnam
        struct packet *result;
        int ret;
 
-       packet = packet_create("release_buffer", "isss", type, provider_name(), pkgname, id);
+       packet = packet_create("release_buffer", "iss", type, pkgname, id);
        if (!packet) {
                ErrPrint("Failed to build a packet\n");
                return LB_STATUS_ERROR_FAULT;
@@ -127,7 +127,7 @@ static inline struct fb_info *send_resize_request(enum target_type type, const c
        const char *buffer_id;
        struct fb_info *fb;
 
-       packet = packet_create("resize_buffer", "isssii", type, provider_name(), pkgname, id, w, h);
+       packet = packet_create("resize_buffer", "issii", type, pkgname, id, w, h);
        if (!packet) {
                ErrPrint("Faield to build a packet\n");
                return NULL;