Commit Graph

49131 Commits

Author SHA1 Message Date
Benjamin Jacobs e03d4142ef ecore_x: NULL derefs.
Reviewers: raster, devilhorns

Subscribers: raster, cedric, jpeg

Differential Revision: https://phab.enlightenment.org/D4018
2016-07-11 14:20:00 +09:00
Carsten Haitzler 899d2d957a edje - language emit - handle null language and convert to none
cleans up an emit string being edje,language,(null) if no language is
set. use none instead of (null).
2016-07-11 13:46:21 +09:00
Youngbok Shin 8e93b88175 edje: update a style when a style is added as class's member
Summary:
If there is no member styles when a text_class is updated,
newly added styles can't be updated.
So, newly added styles as member of text_class should be updated.
@fix

Test Plan:
Test case is included.

1. Run "elementary_test -to "font overlay""
2. Press Next button. Check the font size.
3. Press Prev button.
4. Put font_size as 50
5. Click Apply button.
6. Press Next button. Check the font size is not changed.

Reviewers: cedric, tasn, herdsman, raster

Subscribers: jpeg, z-wony, Blackmole

Differential Revision: https://phab.enlightenment.org/D4125
2016-07-11 13:13:39 +09:00
Shilpa Singh 5b791420ea Accessibility: Read pre-edit text.
Summary:
Edje_entry:  In order to read pre-edit characters as well,
send entry_change_info with preedit,changed
Elm_entry: In order to read pre-edit characters as well,
send text to screen reader on preedit,changed.

Test Plan:
1. Change keyboard language to korean/enable prediction
2. Check the reading, text should be read even before committing

Signed-off-by: Shilpa Singh <shilpa.singh@samsung.com>

Reviewers: thiepha, jihoon, cedric, raster

Reviewed By: raster

Subscribers: raster, kimcinoo, cedric, jpeg

Differential Revision: https://phab.enlightenment.org/D4145
2016-07-11 10:36:39 +09:00
Carsten Haitzler cdbd437c5f evas sw generic - handle alloc failures correctly for map struct
map struct allocation was not handled right - we assumed successthen
later checked for failure with an if() after using the ptr. this
should fix CID 1353722
2016-07-09 12:28:05 +09:00
Carsten Haitzler de9277460a elm view list - fix allocation of werong type size
the code allocated a much bigger data struct than needed. it used the
wrong sizeof() type. this fixes CID 1355013
2016-07-09 12:23:06 +09:00
Carsten Haitzler 116993504a edje edit - api was broken with incorrect l r t b ordering for border
this fixes ordering to match everywhere else ANd usage of the get
func. this was broken already where top/bottom border would swap -
plese see coverity scan CID 1355590 for an example of problems this
created.
2016-07-09 12:17:46 +09:00
Carsten Haitzler 3c5ab2b46e evas generic loader raw - fix small coverity complaint
technically we dont handle a failed fwrite to stdout according to
coverity, but this is harmless as it doesnt matter if it fails - the
owner process cares. slave does not. silence CID 1356615
2016-07-09 12:13:40 +09:00
Carsten Haitzler f6d8ae3123 ecore animator - remove dead code
fix CID 1356618
2016-07-09 12:09:41 +09:00
Carsten Haitzler 3990032330 evas textblock - fix deref before null check shown by coverity
fixes CID 1356927
2016-07-09 12:07:10 +09:00
Carsten Haitzler 9c80c2b070 elput - simplify and fix handling of string overflow with xdg runtimedir
this addresses an issue pointed to by CID 1357168
2016-07-09 12:01:51 +09:00
Carsten Haitzler 870c81fd72 elput - remove pointless code in handling touch event
i commented the pointless code out. this fixes CID 1357145
2016-07-09 11:57:30 +09:00
Carsten Haitzler 891ae89d16 elm atspi bridge - fix resource leak
previous fix brought out a new leak not found before by coverity.

fix CID 1353602
2016-07-09 11:49:59 +09:00
Carsten Haitzler a9025a3e4f evas gl generic - fix ector end to place return in right var
fixes new CID 1357394
2016-07-09 11:44:35 +09:00
Carsten Haitzler f7ea9373f6 edje_cc - fix new coverity issue
fix CID 1357395
2016-07-09 11:44:35 +09:00
Cedric Bail 1be63c1b66 edje: fix build without ephysics.
T4050
2016-07-08 14:16:39 -07:00
Jee-Yong Um 399bb5a0ba edje: add a helper for recursive searching in hash
Summary:
If color class of an edje part is defined as "aaa/bbb/ccc/ddd",
edje will search for color class by the following sequence.
   "aaa/bbb/ccc/ddd"
   "aaa/bbb/ddd"
   "aaa/ddd"
   "ddd"
So, without additional lookup table, edje classes (color, text, size)
can have the functionality like inheritance.

Reviewers: jpeg, raster, cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D4127

Signed-off-by: Cedric Bail <cedric@osg.samsung.com>
2016-07-08 10:50:20 -07:00
perepelits.m 52df6171e9 edje_player: create scene in the edje_player if there are some 3D-components
Summary: check if the size of scene is bigger than 0x0 and build 3D scene in the edje_player in this case and use "opengl_x11"

Reviewers: Hermet, jpeg, cedric

Subscribers: NikaWhite, Oleksander, artem.popov, cedric, jpeg

Differential Revision: https://phab.enlightenment.org/D4041

Signed-off-by: Cedric Bail <cedric@osg.samsung.com>
2016-07-08 10:49:49 -07:00
Woochan Lee 04eb9d2ed5 spinner: fix entry align of spinner.
Summary:
Spinner's entry align changed to center.
(Makes same align with spinner's text.)

Test Plan: elementary_test

Reviewers: jpeg

Subscribers: cedric, jpeg

Differential Revision: https://phab.enlightenment.org/D4147

Signed-off-by: Cedric Bail <cedric@osg.samsung.com>
2016-07-08 10:04:00 -07:00
Chris Michael cb2ab38421 elput: Fix horrid typo
Not sure how this happened, but use a ; not a : ;)

Signed-off-by: Chris Michael <cp.michael@samsung.com>
2016-07-08 08:54:45 -04:00
Chris Michael ee72059219 elput: Fix insecure temporary file
Fix using mkstemp directly without securely setting umask first by
making use of eina_file_mkstemp which does set unmask.

Fixes Coverity CID1357165

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>
2016-07-08 08:51:54 -04:00
Chris Michael b2a8ef3d13 evas-wayland-shm: Fix Insecure temporary file
Fix using mkstemp directly without securely setting umask first by
making use of eina_file_mkstemp function which does set umask.

Fixes Coverity CID1357164

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>
2016-07-08 08:51:54 -04:00
Wonki Kim 19b2915863 elm: Fix potential null point dereferences.
Summary:
If every assigning logics to enginelist is not performed some how,
system will crash because the first parameter of strcmp is null.

Test Plan: N/A

Reviewers: Hermet

Reviewed By: Hermet

Subscribers: cedric, Hermet, raster, jpeg

Differential Revision: https://phab.enlightenment.org/D4149
2016-07-08 21:28:18 +09:00
Hermet Park 9873358f68 edje: fix the map state issue.
While the part state switching, map won't gonna be disabled again,
due to the incorrect state checking.

If there param1 indicates the current state with map off.
then it should disable the map obviously.
But It assumed the param1 is the previous state, not the current!,
so the map didn't go disabled state.

It was intended to avoid the duplicated map state setting.
So to keeping the intention, now it disable the map, only if
the map is actually enabled.

So both are fine.

@fix
2016-07-08 21:22:44 +09:00
Carsten Haitzler 25c63f726e eina thread - fix window build with sigprocmask
fixes T4048
2016-07-08 21:11:21 +09:00
Chris Michael 5a482e9af4 ecore-drm2: Perform input device calibration based on output name
When a new input device gets added, device calibration may need to be
performed. In order for that to be done properly, we need to know
which output this input device is associated with. This patch makes a
function call to Elput in order to retrieve that output name and
perform the proper calibration.

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>
2016-07-08 08:05:20 -04:00
Chris Michael e0cde40475 elput: Add API function to return output name associated with input
device

This patch adds a new API function which we can call from Ecore_Drm2
which will return the name of an output which is associated with a
given input device. This output name can then be used to find a
matching output, and perform any device calibration that is needed.

@feature

Signed-off-by: Chris Michael <cp.michael@samsung.com>
2016-07-08 08:05:14 -04:00
Carsten Haitzler eb1650d5f2 emotion xine module - silence coverity
silence CID 1357140 as its harmless to not check fcntl here, but dont
confuse coverity.
2016-07-08 19:15:10 +09:00
Carsten Haitzler a510349d00 emotion vlc module - fix potential divide by 0
fix CID 1357146
2016-07-08 19:12:41 +09:00
Carsten Haitzler 4354712ccb emotion xine module - silence coverity complaint
getting the fd from a valid fd handler wont return < 0 but in theory
it can, so just be silent coverity

silence CID 1357155
2016-07-08 19:06:57 +09:00
Carsten Haitzler fe9fb7bcf2 emotion xine module - fix minor complaints but not bugs
silence CID 1357156 , 1357172
2016-07-08 19:02:50 +09:00
Carsten Haitzler 46b97aba78 evas module load - make coverity happy but not a bug
silence CID 1357364
2016-07-08 18:59:21 +09:00
Carsten Haitzler b1d9cf8d8d edje cc mo encoding leak - ensure data is freed when setting up encoding
this should fix CID 1306604
2016-07-08 18:53:11 +09:00
Carsten Haitzler 33880a3d7c elm_prefs_cc - clean up return with added brackets to be clear
it seems coverity gets confused with the sizeof(c) / sizeof(type). add
() hoping it will silence it. this is related to:

CID 1353600 and 1353599
2016-07-08 18:33:02 +09:00
Carsten Haitzler 96958909ba elm_prefs_cc - fix correct struct size calculation
use correct struct type even though previous struct was the same size
and it wasnt an actual bug - but a potential one.
2016-07-08 18:24:06 +09:00
Carsten Haitzler e3b6b74aa6 eio model - fix leak found by coverity
fix CID 1355012
2016-07-08 18:15:05 +09:00
Carsten Haitzler 3c793f98a9 gl generic ector leak - fix not using return value of eng_image_data_put
this fixes CID 1339963

@fix
2016-07-08 17:13:17 +09:00
Carsten Haitzler f4568beab6 ecotr gl - fix leak found by coverity
fix CID 1347412
2016-07-08 17:08:37 +09:00
Carsten Haitzler eb4ee72a23 elm map - fix coverity found leak in xml parsing
fix CID 1352814
2016-07-08 17:00:48 +09:00
Carsten Haitzler 9b05497d3c elm map - fix mem leak in xml parsing found by coverity
fix CID 1352815
2016-07-08 16:59:27 +09:00
Carsten Haitzler 1ff099a539 atspi bridge - fix leak on error found by coverity
fix CID 1353602
2016-07-08 16:00:29 +09:00
Carsten Haitzler bb34df4b47 atspi bridge - fix leak on error found by coverity
fix CID 1353604
2016-07-08 16:00:29 +09:00
Carsten Haitzler 2ef7d310d0 edje cc - fix leake found by coverity
fixes CID 1355586
2016-07-08 16:00:29 +09:00
Carsten Haitzler 2fdb465934 edje cc - fix leak of name string on edje compilation time
fixes CID 1355587
2016-07-08 16:00:29 +09:00
Carsten Haitzler bfbb7b15f2 edje edit - fix leaks spotted by coverity
fixes CID 1356630
2016-07-08 16:00:29 +09:00
Carsten Haitzler 4bc6ba4ff0 elm cnp - wl - fix memory leak
fix memory leak specified in CID 1357160 , 1357159 , 1357158 , 1357157
2016-07-08 16:00:29 +09:00
Jean-Philippe Andre 85d45ee9b1 elm: Fix cnp crash on WL
See D4144.

Original author: @slotus.lee

SEG_FAULT happens when the object which has selection is deleted,
and new selection is done at another object.
Reason: loss_cb is not removed when the object which has selection is deleted.
When new selection is set for new object, the loss_cb is called for deleted
object. As result, SEG_FAULT happens.
This issue was also happened in X11 (https://phab.enlightenment.org/D2763)

Test plan:
(on wayland environment) Run elementary test, open Entry, do selection,
close Entry window, open Entry one more time, do selection.
2016-07-08 14:34:26 +09:00
Jean-Philippe Andre 86dc7e0c87 evas: Fix warning about unused function
I kept it here for the record. It was useful, could be useful
again.
2016-07-08 12:27:29 +09:00
Jean-Philippe Andre 73c96c0136 eo: Fix warnins (return NULL instead of bool) 2016-07-08 12:27:29 +09:00
Jean-Philippe Andre b32a8d3a0f ecore: Fix warning about err_no
Double warning, yay:
 explicitly assigning value of variable of type 'int' to itself
 variable 'err_no' is uninitialized when used here

See 1abbfdd1f7
2016-07-08 12:27:29 +09:00